On Thu, 14 Sep 2023 21:15:48 GMT, Mandy Chung <mch...@openjdk.org> wrote:
>> src/java.base/share/classes/java/lang/StackStreamFactory.java line 544: >> >>> 542: return walker.estimateDepth() == 0 >>> 543: ? SMALL_BATCH >>> 544: : Math.min(walker.estimateDepth() + >>> RESERVED_ELEMENTS, LARGE_BATCH_SIZE); >> >> Without the >> `Math.max(walker.estimateDepth()+RESERVED_ELEMENTS, MIN_BATCH_SIZE)` >> for estimateDepth = 1, I believe this will now return 2, where previously it >> returned 3. >> Is that OK? > > yes as it's asked by the user. It will fetch the second batch if it walks > more than 1 frame. Sounds good, thanks ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/15749#discussion_r1326532469
