Skip to content

Commit

Permalink
Run MVU tests in release mode (babelfish-for-postgresql#2113)
Browse files Browse the repository at this point in the history
Task: BABEL-4449
Signed-off-by: Nirmit Shah <[email protected]>
  • Loading branch information
shah-nirmit authored and Jason Teng committed Dec 28, 2023
1 parent 72da14a commit 2ec266a
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 2 deletions.
10 changes: 9 additions & 1 deletion .github/composite-actions/build-modified-postgres/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,10 @@ inputs:
description: 'Tap Tests Enabled'
required: no
default: no
release_mode:
description: 'Build in Release Mode'
required: no
default: no

runs:
using: "composite"
Expand Down Expand Up @@ -42,7 +46,11 @@ runs:
if [[ ${{inputs.tap_tests}} == "yes" ]]; then
./configure CC='ccache gcc' --prefix=$HOME/${{ inputs.install_dir }}/ --with-python PYTHON=/usr/bin/python3.8 --enable-cassert CFLAGS="-ggdb" --with-libxml --with-uuid=ossp --with-icu --enable-tap-tests --with-gssapi
else
./configure CC='ccache gcc' --prefix=$HOME/${{ inputs.install_dir }}/ --with-python PYTHON=/usr/bin/python3.8 --enable-cassert CFLAGS="-ggdb" --with-libxml --with-uuid=ossp --with-icu
if [[ ${{inputs.release_mode}} == "yes" ]]; then
./configure CC='ccache gcc' --prefix=$HOME/${{ inputs.install_dir }}/ --with-python PYTHON=/usr/bin/python3.8 CFLAGS="-ggdb -O2" --with-libxml --with-uuid=ossp --with-icu
else
./configure CC='ccache gcc' --prefix=$HOME/${{ inputs.install_dir }}/ --with-python PYTHON=/usr/bin/python3.8 --enable-cassert CFLAGS="-ggdb" --with-libxml --with-uuid=ossp --with-icu
fi
fi
make -j 4 2>error.txt
make install
Expand Down
10 changes: 10 additions & 0 deletions .github/workflows/major-version-upgrade.yml
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,7 @@ jobs:
uses: ./.github/composite-actions/build-modified-postgres
with:
install_dir: ${{env.NEW_INSTALL_DIR}}
release_mode: 'yes'

- name: Copy ANTLR
run: cp "/usr/local/lib/libantlr4-runtime.so.4.9.3" ~/${{env.NEW_INSTALL_DIR}}/lib/
Expand Down Expand Up @@ -174,6 +175,15 @@ jobs:
if: always() && steps.setup-new-datadir.outcome == 'success'
uses: ./.github/composite-actions/run-pg-upgrade

- name: Disable TDS fault injection tests in release mode
id: disable-fault-injection
if: always() && steps.setup-new-datadir.outcome == 'success'
run: |
cd test/JDBC/
echo 'ignore#!#babel_tds_fault_injection' >> jdbc_schedule
echo 'ignore#!#tds_faultinjection' >> jdbc_schedule
echo jdbc_schedule
- name: Run JDBC Tests
id: jdbc
timeout-minutes: 60
Expand Down
2 changes: 1 addition & 1 deletion contrib/babelfishpg_tsql/src/pltsql_coerce.c
Original file line number Diff line number Diff line change
Expand Up @@ -1348,7 +1348,7 @@ select_common_type_for_isnull(ParseState *pstate, List *exprs)
static int32
tsql_select_common_typmod_hook(ParseState *pstate, List *exprs, Oid common_type)
{
int32 max_typmods;
int32 max_typmods=0;
ListCell *lc;
common_utility_plugin *utilptr = common_utility_plugin_ptr;

Expand Down

0 comments on commit 2ec266a

Please sign in to comment.