[THRIFT-3269] error handling issue in SimpleServer

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: Go - Library
• Assignee: Unassigned
• Reporter: Jens Geyer
• Created: 26/Jul/15 00:12
• Updated: 26/Jul/15 00:26

Description

There is error handling issue in SimpleServer, which supresses errors instead of properly handling them. If the comparingly simple fix is applied, one of the Xception test fails all of a sudden.

2015/07/26 01:52:32 error processing request: TException
2015/07/26 01:52:32 error processing request: TException
--- FAIL: TestTJSONProtocol (0.06s)
        thrifttest_driver.go:224: TestException Xception failed
2015/07/26 01:52:32 error processing request: TException
2015/07/26 01:52:32 error processing request: TException
--- FAIL: TestTBinaryProtocol (0.08s)
        thrifttest_driver.go:224: TestException Xception failed
2015/07/26 01:52:32 error processing request: TException
2015/07/26 01:52:32 error processing request: TException
--- FAIL: TestTCompactProtocol (0.07s)
        thrifttest_driver.go:224: TestException Xception failed
FAIL
FAIL    tests   0.398s