Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[YSQL] Test fk-contention in isolation regress suite fails (the test hasn't yet been added) #12325

Open
pkj415 opened this issue Apr 29, 2022 · 0 comments
Labels
area/ysql Yugabyte SQL (YSQL) kind/enhancement This is an enhancement of an existing feature pg-compatibility Label for issues that result in differences b/w YSQL and Pg semantics priority/medium Medium priority issue

Comments

@pkj415
Copy link
Contributor

pkj415 commented Apr 29, 2022

Jira Link: DB-496

Description

Check fk-contention.spec for the test and fk-contention.out for the expected output. We fail in YSQL because the UPDATE conflicts with the lock taken by the INSERT. Avoiding the conflict will help us improve performance in such cases and also achieve same semantics as Pg.

To quickly repro it, add "test: fk-contention" to the yb_pg_isolation_schedule, comment out other tests and then run either -

  1. To run against locally running cluster, inside yugabyte-db repo:
> mkdir -p pgregress_output/yb_pg_isolation_schedule

> YB_PG_REGRESS_RESULTSFILE_POSTPROCESS_CMD=`pwd`/python/yugabyte/pg_regress_postprocess_output.py ./build/latest/postgres_build/src/test/isolation/pg_isolation_regress --bindir=./build/latest/postgres/bin --use-existing --inputdir=./src/postgres/src/test/isolation --outputdir=./pgregress_output/yb_pg_isolation_schedule --schedule=./src/postgres/src/test/isolation/yb_pg_isolation_schedule --host=127.0.0.1 --port=5433 --user=yugabyte --dbname=yugabyte
  1. Via java test (slower): ./yb_build.sh --java-test org.yb.pgsql.TestPgIsolationRegress#isolationRegress
@pkj415 pkj415 added area/ysql Yugabyte SQL (YSQL) status/awaiting-triage Issue awaiting triage labels Apr 29, 2022
@pkj415 pkj415 changed the title [YSQL] Test fk-contention in isolation regress suite fails [YSQL] Test fk-contention in isolation regress suite fails (the test hasn't yet been added) Apr 29, 2022
@pkj415 pkj415 added the pg-compatibility Label for issues that result in differences b/w YSQL and Pg semantics label May 24, 2022
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels May 24, 2022
@yugabyte-ci yugabyte-ci added kind/new-feature This is a request for a completely new feature and removed kind/bug This issue is a bug status/awaiting-triage Issue awaiting triage labels Jul 27, 2022
@yugabyte-ci yugabyte-ci added kind/enhancement This is an enhancement of an existing feature and removed kind/new-feature This is a request for a completely new feature labels Aug 18, 2022
@yugabyte-ci yugabyte-ci added priority/high High Priority priority/medium Medium priority issue and removed priority/medium Medium priority issue priority/high High Priority labels Aug 31, 2023
@yugabyte-ci yugabyte-ci added priority/medium Medium priority issue and removed priority/high High Priority labels Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ysql Yugabyte SQL (YSQL) kind/enhancement This is an enhancement of an existing feature pg-compatibility Label for issues that result in differences b/w YSQL and Pg semantics priority/medium Medium priority issue
Projects
None yet
Development

No branches or pull requests

4 participants