keith-turner commented on issue #948: Investigate parallelizing TransactionImpl.readUnread() URL: https://github.com/apache/fluo/issues/948#issuecomment-544969648 > Is this still valid? Yes. > I was looking at `Map<RowColumn, Bytes> get(Collection<RowColumn> rowColumns)` and `get(Collection<Bytes> rows, Set<Column> columns)` as options but they don't have the Consumer<Entry<Key, Value>> locksSeen parameter. I feel like this is the right path. Looking inside `get(Collection<RowColumn> rowColumns)` there is a `ParallelSnapshotScanner`, I wonder if this could be used. Create a parallel snapshot scanner and then read all the data from it. Can pass `locksSeen` to the constructor of `ParallelSnapshotScanner`. > Another option would be something like the following, which just runs all the getImpl call concurrently. I am wary of using `CompletableFuture.runAsync()` for operations that do I/O because it runs in a JVM wide shared thread pool. If the Fluo client library does this it could impact user code that is also using `runAsync()`
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
