babelfish-for-postgresql / babelfish_extensions / 15585066740
76%
BABEL_6_X_DEV: 78%

LAST BUILD BRANCH: BABEL-5975-PR1-full
DEFAULT BRANCH: BABEL_6_X_DEV
Ran
Jobs 1
Files 119
Run time 100min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 75.403% (+0.008%) from 75.395%
15585066740

push

github

web-flow 
Fix loading of data for TEXT column (NULL allowed) if primary key is defined using non-default collation from PG endpoint  for logical database (#3769)

If the babelfish table has a column whose type is anything other than text/ntext (eg: varchar, nvarchar etc) and the collation of the column is NOT same as babelfish database/server level collation and if the query is executed from PSQL endpoint, it will throw error saying it can not decide which collation to use.

If the same query is executed from TDS endpoint, it runs successfully.

This is because when the query is executed from PSQL endpoint -- the collation of varchar is server level collation. During resolution of collation at the parent node, the collation of left and right operand does NOT match and none of their collation matches with CLUSTER_COLLATION_OID (this returns DEFAULT_COLLATION_OID as the dialect is PG)

The query is working from TDS endpoint -- the collation of varchar is database level collation. During resolution of collation at the parent node, the collation of left and right operand does NOT match and collation of right operand matches with CLUSTER_COLLATION_OID (this returns database level collation as the dialect is TSQL)

This commit handles the case when the connection is from PG endpoint and psql_logical_babelfish_db_name GUC is set -- the case for logical/TSQL database. If both the aforementioned conditions are met, we will return babelfish server level collation irrespective of dialect (initially it was only for TDS connection).

Task: BABEL-5077

Signed-off-by: Shameem Ahmed shmeeh@amazon.com

48605 of 64460 relevant lines covered (75.4%)

321529.7 hits per line

Coverage Regressions

Lines Coverage ∆ File
156
74.24
 0.16% contrib/babelfishpg_tsql/src/collation.c
83
79.02
 0.32% contrib/babelfishpg_common/src/collation.c
21
84.93
 0.0% contrib/babelfishpg_tsql/src/hooks.c
Jobs
ID Job ID Ran Files Coverage
1 15585066740.1 119
75.4
 GitHub Action Run
Source Files on build 15585066740