Manno15 commented on a change in pull request #2273:
URL: https://github.com/apache/accumulo/pull/2273#discussion_r710272098
##########
File path:
core/src/main/java/org/apache/accumulo/core/clientImpl/ThriftTransportPool.java
##########
@@ -654,6 +655,31 @@ public void consumeBuffer(int len) {
}
}
+ @Override
+ public TConfiguration getConfiguration() {
+ return wrappedTransport.getConfiguration();
+ }
+
+ @Override
+ public void updateKnownMessageSize(long size) throws TTransportException {
+ try {
+ ioCount++;
Review comment:
Might have something to do with `// when ioCount is odd, it means I/O is
currently happening`. I don't know if it factors there but I can see the case
to make it odd before an operation, and when done, make it even again.
EDIT: Realize the comments above explained this too. In checkForStuckIO()
```
// Tried to call System.currentTimeMillis() and Thread.currentThread()
before every io
// operation.... this dramatically slowed things down. So switched to
// incrementing a counter before and after each io operation.
if ((ioCount & 1) == 1) {
// when ioCount is odd, it means I/O is currently happening
if (ioCount == lastIoCount) {
```
--
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]
