[HIVE-14902] HiveServer2: Cleanup code which checks for ThriftJDBCSerde usage

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.1.0
• Fix Version/s: 2.1.1, 2.2.0
• Component/s: HiveServer2, JDBC, ODBC
• Target Version/s: 2.1.1
• Hadoop Flags: Reviewed
• Assignee: Ziyang Zhao
• Reporter: Vaibhav Gumashta
• Created: 06/Oct/16 08:09
• Updated: 08/Dec/16 14:10
• Resolved: 18/Nov/16 22:56

Description

As part of HIVE-14039, we simplified the condition checking by setting a boolean in SessionState (from HiveSessionImpl) and then using that in other layers to decide whether we should use ThriftJDBCSerde. However, there are some parts of the code (e.g. FileSinkOperator.closeOp, Driver.getResults, SQLOperation.getNextRowSet) where the condition checking is still complicated - we should refactor that.

Issue Links

• links to: Review Board-ziyangz