You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/12/29 17:02:34 UTC

[GitHub] [accumulo] DomGarguilo opened a new issue #2402: Explore moving `Scanner.setBatchSize()` to `ScannerBase`

DomGarguilo opened a new issue #2402:
URL: https://github.com/apache/accumulo/issues/2402


   **Is your feature request related to a problem? Please describe.**
   BatchScanner has very few options for controlling the amount of data retrieved. Propose moving setBatchSize to ScannerBase to allow for this configuration option to be applied to BatchScanners as well as scanners.
   
   **Describe the solution you'd like**
   Move `Scanner.setBatchSize()` to `ScannerBase`
   
   **Additional context**
   This ticket is based off of Jira ticket [ACCUMULO-2001](https://issues.apache.org/jira/browse/ACCUMULO-2001). It was also mentioned on the ticket:
   
   > Before moving to a common base class, the naming of this method should be reconsidered... as the "Batch" in "BatchScanner" (requests are batched, according to the hosting tserver) means something completely different than the "Batch" in "setBatchSize" (results are fetched in batches).
   


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii commented on issue #2402: Explore moving `Scanner.setBatchSize()` to `ScannerBase`

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #2402:
URL: https://github.com/apache/accumulo/issues/2402#issuecomment-1003150803


   One alternate name for "BatchScanner" was "ParallelScanner". That would be a big change, but it would make more sense to leave "setBatchSize" with the current name, if the class was named "ParallelScanner".


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org