<i18n dev> RFR: 8333396: Performance regression of DecimalFormat.format [v8]

[Naoto Sato]

On Thu, 20 Jun 2024 18:58:27 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> lingjun-cg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8333396: Performance regression of DecimalFormat.format
>
> I did not mean to introduce a public API for this change (if we do, the fix cannot be backported). I thought we could define a package private one, but based on your observation, it may get messier... So I agree that falling back to `StringBuf` is the way to go, IMO.

> So, considering all the information given, is it enough to start our new review process? @naotoj @liach @justin-curtis-lu

Well, I was suggesting the same buffer proxying for other Format classes than NumberFormat subclasses, such as DateFormat so that they would have the same performance benefit. Would you be willing to do that too?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/19513#issuecomment-2189821715