ctubbsii commented on code in PR #3124:
URL: https://github.com/apache/accumulo/pull/3124#discussion_r1103415648
##########
core/src/main/java/org/apache/accumulo/core/clientImpl/ActiveScanImpl.java:
##########
@@ -55,6 +55,7 @@ public class ActiveScanImpl extends ActiveScan {
private Map<String,Map<String,String>> ssio;
private String user;
private Authorizations authorizations;
+ private String correlationId;
Review Comment:
> I think a client correlationId should be the goal, this is a step that
would not be incompatible with that.
I think adding the APIs to ScannerBase would need to be rolled back if we're
going to make it client-wide. It doesn't make sense to add it there if we're
going to change it to be added to AccumuloClient later.
--
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]
