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

Log work

Agile Board

Rank to Top

Rank to Bottom

Bulk Copy Attachments

Bulk Move Attachments

Voters

Watch issue

Watchers

Convert to Issue

Move

Link

Clone

Labels

Update Comment Author

Replace String in Comment

Update Comment Visibility

Delete Comments

XML

Word

Printable

JSON

Details

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
Labels:
None

Target Version/s:

2.1.1
Hadoop Flags:

Reviewed

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.