[THRIFT-4943] org.apache.thrift.transport.TTransportException: null

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 0.12.0
• Fix Version/s: 0.13.0
• Component/s: Java - Library
• Assignee: Unassigned
• Reporter: imic
• Created: 28/Aug/19 07:48
• Updated: 24/Sep/19 22:16
• Resolved: 24/Sep/19 22:16

Description

libthrift ver 0.12.0 TThreadPoolServer.java  line 315

code in TODO B should put in TODO A
 

 

//代码占位符
      } catch (TException tx) {

        // TODO A.this code is not correct

        LOGGER.error("Thrift error occurred during processing of message.", tx);
      } catch (Exception x) {
        // We'll usually receive RuntimeException types here
        // Need to unwrap to ascertain real causing exception before we choose to ignore
        Throwable realCause = x.getCause();
        // Ignore err-logging all transport-level/type exceptions
        if ((realCause != null && realCause instanceof TTransportException)
        
            // TODO B.this code should put in area TODO A，TTransportException is subclass of TException ，TODO A already handled TTransportException, so this code is nessesary . libthrift ver 0.11.0 is collect

            || (x instanceof TTransportException)) {
          if (LOGGER.isDebugEnabled()) {
            // Write to debug, just in case the exception gets required
            LOGGER
                .debug("Received TTransportException during processing of message, ignoring: ", x);
          }
        } else {
          // Log the exception at error level and continue
          LOGGER.error("Error occurred during processing of message.", x);
        }
      }

 
 
 
 
 

Issue Links

• duplicates: THRIFT-4805 Suppress excessive logging of SASL TTransportExceptions in case of END_OF_FILE
• relates to: THRIFT-4805 Suppress excessive logging of SASL TTransportExceptions in case of END_OF_FILE