Apache9 commented on code in PR #5119:
URL: https://github.com/apache/hbase/pull/5119#discussion_r1143501920
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/AbstractFSWAL.java:
##########
@@ -298,27 +298,34 @@
final Comparator<Path> LOG_NAME_COMPARATOR =
(o1, o2) -> Long.compare(getFileNumFromFileName(o1),
getFileNumFromFileName(o2));
- private static final class WalProps {
+ private static final class WALProps {
/**
* Map the encoded region name to the highest sequence id.
* <p/>
* Contains all the regions it has an entry for.
*/
- public final Map<byte[], Long> encodedName2HighestSequenceId;
+ final Map<byte[], Long> encodedName2HighestSequenceId;
/**
* The log file size. Notice that the size may not be accurate if we do
asynchronous close in
* sub classes.
*/
- public final long logSize;
+ final long logSize;
/**
* The nanoTime of the log rolling, used to determine the time interval
that has passed since.
*/
- public final long rollTimeNs;
+ final long rollTimeNs;
Review Comment:
We could but not necessary? The class itself is already private...
--
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]
