Refactoring StringBuffer to StringBuilder

[ochaloup]

Hi,

I just hit this article

https://vanilla-java.github.io/2017/04/13/String-Buffer-and-how-hard-it-is-to-get-rid-of-legacy-code.html

I interested if you give +1 on changing StringBuffer for StringBuilder at places where it's used in Narayana code. I expect it's an optional task but for case of no objections, I would create a jira to track the idea.

What do you think?

Thanks

Ondra

[tomjenkinson]

I am sceptical as to whether it would have any material effect on Narayana performance or runtime profile. I don't expect that as a normal 2PC we would actually use StringBuffer at all? Do you see any examples of that?

[mmusgrov]

We have Performance Gates for accepting performance fixes in Narayana for accepting performance related fixes. If you raise a PR and check the PERF profile you can quantify the effect of your enhancements.

[ochaloup]

tomjenkinson I see, agree. I think performance probably would not get much better. I feel it more like refactoring task where the aim is to have a better code.

mmusgrov that's the good point, I think going that way is the ideal.