omkreddy merged PR #16513:
URL: https://github.com/apache/kafka/pull/16513
--
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
apoorvmittal10 commented on PR #16513:
URL: https://github.com/apache/kafka/pull/16513#issuecomment-2206307430
@AndrewJSchofield @omkreddy The build passed with unrelated tests failure.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
apoorvmittal10 commented on code in PR #16513:
URL: https://github.com/apache/kafka/pull/16513#discussion_r1663843416
##
server/src/main/java/org/apache/kafka/server/share/ShareSessionCache.java:
##
@@ -41,10 +41,10 @@ public class ShareSessionCache {
private long numPartit
apoorvmittal10 commented on code in PR #16513:
URL: https://github.com/apache/kafka/pull/16513#discussion_r1663835968
##
core/src/test/java/kafka/server/share/SharePartitionManagerTest.java:
##
@@ -1246,26 +1250,40 @@ public void testCloseSharePartitionManager() throws
Exceptio
AndrewJSchofield commented on code in PR #16513:
URL: https://github.com/apache/kafka/pull/16513#discussion_r1663821656
##
server/src/main/java/org/apache/kafka/server/share/ShareSessionCache.java:
##
@@ -41,10 +41,10 @@ public class ShareSessionCache {
private long numPart
adixitconfluent commented on code in PR #16513:
URL: https://github.com/apache/kafka/pull/16513#discussion_r1663802402
##
core/src/test/java/kafka/server/share/SharePartitionManagerTest.java:
##
@@ -1246,26 +1250,40 @@ public void testCloseSharePartitionManager() throws
Excepti
apoorvmittal10 commented on PR #16513:
URL: https://github.com/apache/kafka/pull/16513#issuecomment-2205430052
The context: https://github.com/apache/kafka/pull/16456/files#r1662556294
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
apoorvmittal10 opened a new pull request, #16513:
URL: https://github.com/apache/kafka/pull/16513
The release API exposed Partitions which should be an internal
implementation detail for `releaseAcquiredRecords` API. Also lessen the scope
for cached topic partitons method as it's not needed