babelfish-for-postgresql / babelfish_extensions / 7694660890
76%
BABEL_5_X_DEV: 77%

LAST BUILD BRANCH: fix_script_injection
DEFAULT BRANCH: BABEL_5_X_DEV
Ran
Jobs 1
Files 113
Run time 12s
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: 72.057% (+0.02%) from 72.042%
7694660890

push

github

web-flow 
Fix issues with datatypes for Logical Replication (#2268)

This commit fixes following issues:

1. (Var)binary columns are not replicated correctly since input function considers
the value as a raw ASCII string although the value is actually already in binary format.
Fixed this by using IsLogicalWorker() with existing TSQLHexConstTypmod
check in varbinaryin function to correctly take binary input sent by the publisher.
2. SQL_VARIANT columns are replicated correctly as far as the data contents are
concerned, but the meta data is lost in the process which is because sql_variant output
function returns the value in the respective datatype in string format which when fed
back to the sql_variant input function gets treated as a varchar value.
Fixed this by replicating sql_variant values as bytea. We will use MyReplicationSlot
and MyWalSnd in sqlvariantout function on publisher and IsLogicalWorker() in
sqlvariantin function on subscriber to identify if we are in logical replication context.
3. On the subscriber side, IsLogicalWorker() is sufficient for native PG applyworker
but will not work with external providers like pglogical, so will rely on SessionReplicationRole
being replica since most of the providers seem to set this GUC.

Additionally, this adds support to run logical replication tests in github action along with
JDBC framework.

Task: BABEL-3427, BABEL-4049
Signed-off-by: Rishabh Tanwar ritanwar@amazon.com

40187 of 55771 relevant lines covered (72.06%)

60230.18 hits per line

Jobs
ID Job ID Ran Files Coverage
1 7694660890.1 113
72.06
 GitHub Action Run
Source Files on build 7694660890