billoley commented on code in PR #2790:
URL: https://github.com/apache/accumulo/pull/2790#discussion_r909689406
##########
server/tserver/src/main/java/org/apache/accumulo/tserver/scan/LookupTask.java:
##########
@@ -87,23 +85,23 @@ public void run() {
Key partNextKey = null;
boolean partNextKeyInclusive = false;
- Iterator<Entry<KeyExtent,List<Range>>> iter =
session.queries.entrySet().iterator();
+ // Iterator<Entry<KeyExtent,List<Range>>> iter =
session.queries.entrySet().iterator();
+ log.error("number of extents: " + session.queries.size() + " -- " +
session.queries.keySet());
Review Comment:
I left those lines in by accident & will remove them. I'm thinking of
adding session.queries to a LinkedList<Pair<KeyExtent, List<Range>>> so that we
can add to the back and remove from the front instead of using
session.queries.iterator().next() when we're not really iterating.
The fact that partScan/partNextKey/partNextKeyInclusive only report back one
partial scan when there could be multiple partial scans still seems
problematic, but that may be a topic for a subsequent issue,
--
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]
