cradal commented on a change in pull request #1651:
URL: https://github.com/apache/accumulo/pull/1651#discussion_r526260981
##########
File path:
core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletsMetadata.java
##########
@@ -79,6 +79,15 @@
private boolean checkConsistency = false;
private boolean saveKeyValues;
private TableId tableId;
+ private AccumuloClient _client;
+
+ Builder(AccumuloClient client) {
+ this._client = client;
+ }
+
+ Builder() {
+
+ }
Review comment:
One issue with this is that there are ~15 classes that already use the
pre-existing no-parameter constructor. Perhaps I should find a different way to
pass the AmpleImpl Accumulo client into the TabletsMetadata builder object.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
