jlprat commented on code in PR #14816:
URL: https://github.com/apache/kafka/pull/14816#discussion_r1401847506
##########
server-common/src/main/java/org/apache/kafka/timeline/Snapshot.java:
##########
@@ -24,10 +24,10 @@
* A snapshot of some timeline data structures.
* <br>
* The snapshot contains historical data for several timeline data structures.
- * We use an IdentityHashMap to store this data. This way, we can easily drop
all of
+ * We use an IdentityHashMap to store this data. This way, we can easily drop
all
* the snapshot data.
*/
-class Snapshot {
+public class Snapshot {
Review Comment:
Class is now public, but only read methods are public, so no class from
outside the package can tamper with the data in this data structure
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
