babelfish-for-postgresql / babelfish_extensions / 17639046326

76%

BABEL_5_X_DEV: 77%

push

github

Fixed error when parallel worker tries to access rel with oid > INT32_MAX (#4065) (#4077)

As part of the commit 6da7ff0, we introduced logic to skip permission checking of Babelfish temp table on Parallel
workers. But logic will run into an error if Oid is greated than INT32_MAX because Bitmapset was used to
communicate Oids from leader node to parallel worker. This Bitmapset can only work with signed integer meaning if
Oid value is greater than INT32_MAX then it will be rounded off to negative value in signed integer and Bitmapset will
throw an error that it can't handle negative values.

In order to fix this overflow issue, we create one more bitmapset especially for Oid > INT32_MAX and communicate it
with parallel worker. Construction of bitmapset now would look like this,

if relid <= INT32_MAX
    add relid to temp_relids
else
    add (relid - INT32_MAX - 1) to temp_relids_beyond_int32

Task: BABEL-6079
Signed-off-by: Dipesh Dhameliya <dddhamel@amazon.com>

15 of 19 new or added lines in 1 file covered. (78.95%)

49834 of 65587 relevant lines covered (75.98%)

381657.4 hits per line