[
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) {
// 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
{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}
> 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) {
> // 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)
