EdColeman commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086735011
##########
core/src/main/java/org/apache/accumulo/core/client/ClientSideIteratorScanner.java:
##########
@@ -213,7 +213,7 @@ public ClientSideIteratorScanner(final Scanner scanner) {
smi = new ScannerTranslatorImpl(scanner,
scanner.getSamplerConfiguration());
this.range = scanner.getRange();
this.size = scanner.getBatchSize();
- this.timeOut = scanner.getTimeout(MILLISECONDS);
+ this.retryTimeout = scanner.getTimeout(MILLISECONDS);
this.batchTimeOut = scanner.getTimeout(MILLISECONDS);
Review Comment:
Yes - this is step one with the minimum to fix the parameter checks and a
little renaming to help with resolving the API conflict.
--
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]
