dlmarion commented on code in PR #3124:
URL: https://github.com/apache/accumulo/pull/3124#discussion_r1103195077
##########
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:
> Say that a client reads, possibly updates and then writes the modified or
new data within the scope of one unit of work for the client. When logged
consistently, the correlationId could be used to tie all of the log statements
for that client operation, scan writes,... across servers and processes.
> Right, in that sense it's not a scan correlationId, it's a client
correlationId.
I was just saying that the same correlationId would be used by the client
for those operations. I wasn't proposing a correlationId at the client.
--
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]
