babelfish-for-postgresql / babelfish_extensions / 24593574385

77%

BABEL_6_X_DEV: 78%

Build Type

push

github

Committed by web-flow

Commit Message

Fix for Invalid Token Stream error for simple queries in go driver (#4757)

Simple queries like SELECT 1 AS test were failing on go-sqlcmd v1.9.0 with error: Invalid TDS stream: unknown token type returned: token(165) causing the session to hang. This was because TdsSendRowDescription()was sending colinfo token for regular queries. The go-mssqldb driver does not implement a parser for colinfo token, so it reports the token as unknown and the TDS stream becomes desynchronized, causing the session to hang.

Fix: Guarded SendColInfoToken() with if (relMetaDataInfoList != NIL) so it is only sent when table metadata exists. Also, added a no_browsetable check, when NO_BROWSETABLE is ON, the COLINFO token is sent, otherwise it is not sent.

Task: BABEL-6423

Authored by: Vineetha125 bvineets@amazon.com

Coverage Stats

5 of 0 new or added lines in 0 files covered. (NaN%)

77 existing lines in 3 files now uncovered.

53061 of 69069 relevant lines covered (76.82%)

405502.3 hits per line