milleruntime commented on a change in pull request #2273:
URL: https://github.com/apache/accumulo/pull/2273#discussion_r711197487
##########
File path:
core/src/main/java/org/apache/accumulo/core/util/ThriftMessageUtil.java
##########
@@ -49,7 +50,11 @@ public ThriftMessageUtil(int initialCapacity) {
// TODO does this make sense? better to push this down to the serialize
method (accept the
// transport as an argument)?
this.initialCapacity = initialCapacity;
- this.inputTransport = new TMemoryInputTransport();
+ try {
+ this.inputTransport = new TMemoryInputTransport();
+ } catch (TTransportException e) {
+ throw new RuntimeException(e);
Review comment:
Looking at the code that calls this, it may be better to throw an
IOException here.
--
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]
