EdColeman commented on code in PR #3876:
URL: https://github.com/apache/accumulo/pull/3876#discussion_r1367822257
##########
core/src/main/java/org/apache/accumulo/core/metadata/schema/RootTabletMetadata.java:
##########
@@ -73,10 +77,27 @@ private static class Data {
*/
private final TreeMap<String,TreeMap<String,String>> columnValues;
- public Data(int version, TreeMap<String,TreeMap<String,String>>
columnValues) {
+ private Data(int version, TreeMap<String,TreeMap<String,String>>
columnValues) {
this.version = version;
this.columnValues = columnValues;
}
+
+ public int getVersion() {
+ return version;
+ }
+
+ /**
+ * For external use, return a copy so the original remains immutable.
+ */
+ public TreeMap<String,TreeMap<String,String>> getColumnValues() {
+ return new TreeMap<>(columnValues);
Review Comment:
With refactoring, this method was not needed outside of the class and the
method was removed in e8ae63227647
--
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]
