On Thu, 7 Oct 2021 21:46:47 GMT, Alex Menkov wrote:
> The fix adds "-XX:PerfMaxStringConstLength" argument running target app
> (default is 1024, 8K should be enough for any environments)
Marked as reviewed by lmesnik (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/5858
On Thu, 7 Oct 2021 21:46:47 GMT, Alex Menkov wrote:
> The fix adds "-XX:PerfMaxStringConstLength" argument running target app
> (default is 1024, 8K should be enough for any environments)
Ok. You can proceed with your current fix if you'd like.
-
PR: https://git.openjdk.java.net/j
On Wed, 27 Oct 2021 23:09:09 GMT, Chris Plummer wrote:
> Actually revoking my review for the moment. Do we have any tests that
> currently test the default PerfMaxStringConstLength, and this change could be
> subverting the test by making it so the length is never exceeded?
We don't have any t
On Thu, 7 Oct 2021 21:46:47 GMT, Alex Menkov wrote:
> The fix adds "-XX:PerfMaxStringConstLength" argument running target app
> (default is 1024, 8K should be enough for any environments)
Marked as reviewed by cjplummer (Reviewer).
Actually revoking my review for the moment. Do we have any tes
On Thu, 7 Oct 2021 21:46:47 GMT, Alex Menkov wrote:
> The fix adds "-XX:PerfMaxStringConstLength" argument running target app
> (default is 1024, 8K should be enough for any environments)
Ping.
Need 2nd reviewer
-
PR: https://git.openjdk.java.net/jdk/pull/5858
On Thu, 7 Oct 2021 21:46:47 GMT, Alex Menkov wrote:
> The fix adds "-XX:PerfMaxStringConstLength" argument running target app
> (default is 1024, 8K should be enough for any environments)
LGTM.
Thanks,
Serguei
-
Marked as reviewed by sspitsyn (Reviewer).
PR: https://git.openjdk.j
The fix adds "-XX:PerfMaxStringConstLength" argument running target app
(default is 1024, 8K should be enough for any environments)
-
Commit messages:
- JDK-8274930
Changes: https://git.openjdk.java.net/jdk/pull/5858/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5858
