ChrisAHolland commented on code in PR #15507: URL: https://github.com/apache/kafka/pull/15507#discussion_r1524100301
########## server-common/src/main/java/org/apache/kafka/server/mutable/BoundedList.java: ########## @@ -35,24 +35,12 @@ public class BoundedList<E> implements List<E> { private final int maxLength; private final List<E> underlying; - public static <E> BoundedList<E> newArrayBacked(int maxLength) { - return new BoundedList<>(maxLength, new ArrayList<>()); - } - - public static <E> BoundedList<E> newArrayBacked(int maxLength, int initialCapacity) { Review Comment: I like the idea of migrating `newArrayBacked(int maxLength`) to `newLinkedListBacked(int maxLength)`, this would also help cut down on array list resizing time, as currently this constructor creates a default array list, which is size 10 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org