thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "imic (Jira)" <j...@apache.org>
Subject [jira] [Updated] (THRIFT-4943) org.apache.thrift.transport.TTransportException: null
Date Wed, 28 Aug 2019 07:53:00 GMT

     [ https://issues.apache.org/jira/browse/THRIFT-4943?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

imic updated THRIFT-4943:
-------------------------
    Description: 
libthrift ver 0.12.0 TThreadPoolServer.java  line 315

code in TODO B should put in TODO A
  
{code:java}
  } catch (TException tx) {  } 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); }  }
{code}
 
 
 
 
 
 
 
 
 
 

  was:
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);
        }
      }
 


> org.apache.thrift.transport.TTransportException: null
> -----------------------------------------------------
>
>                 Key: THRIFT-4943
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4943
>             Project: Thrift
>          Issue Type: Bug
>          Components: Java - Library
>    Affects Versions: 0.12.0
>            Reporter: imic
>            Priority: Major
>             Fix For: 0.11.0
>
>
> libthrift ver 0.12.0 TThreadPoolServer.java  line 315
> code in TODO B should put in TODO A
>   
> {code:java}
>   } catch (TException tx) {  } 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); }  }
> {code}
>  
>  
>  
>  
>  
>  
>  
>  
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Mime
View raw message