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

roachtest: pgjdbc failed #39406

Closed
cockroach-teamcity opened this issue Aug 7, 2019 · 8 comments · Fixed by #40481
Closed

roachtest: pgjdbc failed #39406

cockroach-teamcity opened this issue Aug 7, 2019 · 8 comments · Fixed by #40481
Assignees
Labels
C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot.
Milestone

Comments

@cockroach-teamcity
Copy link
Member

SHA: https://github.com/cockroachdb/cockroach/commits/a850466dfc4f8eed9e7f758a61f60b120798410f

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stress instead of stressrace and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
stdbuf -oL -eL \
make stressrace TESTS=pgjdbc PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-maxtime 20m -timeout 10m' 2>&1 | tee /tmp/stress.log

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=1426564&tab=buildLog

The test failed on branch=release-19.1, cloud=gce:
test artifacts and logs in: /home/agent/work/.go/src/github.com/cockroachdb/cockroach/artifacts/20190807-1426564/pgjdbc/run_1
	pgjdbc.go:294,pgjdbc.go:305,test_runner.go:691: 
		Tests run on Cockroach v19.1.3-33-ga850466
		Tests run against pgjdbc REL42.2.6
		5786 Total Tests Run
		5768 tests passed
		18 tests failed
		0 tests passed unexpectedly
		4 tests failed unexpectedly
		0 tests expected failed, but not run
		For a full summary look at the pgjdbc artifacts 
		
		An updated blacklist (pgjdbcBlackList19_1) is available in the artifacts' pgjdbc log

@cockroach-teamcity cockroach-teamcity added this to the 19.2 milestone Aug 7, 2019
@cockroach-teamcity cockroach-teamcity added C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. labels Aug 7, 2019
@jordanlewis jordanlewis assigned rafiss and unassigned andreimatei Aug 7, 2019
@rafiss
Copy link
Collaborator

rafiss commented Aug 7, 2019

Here is a blacklist we could use for 19.1

var pgjdbcBlackList19_1 = blacklist{
  "org.postgresql.test.jdbc2.ClientEncodingTest.setEncodingAscii[allowEncodingChanges=true]": "unknown",
  "org.postgresql.test.jdbc4.ClientInfoTest.testExplicitSetAppNameNotificationIsParsed": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = -infinity, pgType = date, klass = class java.time.LocalDate]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = -infinity, pgType = timestamp with time zone, klass = class java.time.OffsetDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = -infinity, pgType = timestamp, klass = class java.time.LocalDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = -infinity, pgType = timestamp, klass = class java.time.OffsetDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = infinity, pgType = date, klass = class java.time.LocalDate]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = infinity, pgType = timestamp with time zone, klass = class java.time.OffsetDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = infinity, pgType = timestamp, klass = class java.time.LocalDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = FORCE, expr = infinity, pgType = timestamp, klass = class java.time.OffsetDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = -infinity, pgType = date, klass = class java.time.LocalDate]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = -infinity, pgType = timestamp with time zone, klass = class java.time.OffsetDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = -infinity, pgType = timestamp, klass = class java.time.LocalDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = -infinity, pgType = timestamp, klass = class java.time.OffsetDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = infinity, pgType = date, klass = class java.time.LocalDate]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = infinity, pgType = timestamp with time zone, klass = class java.time.OffsetDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = infinity, pgType = timestamp, klass = class java.time.LocalDateTime]": "unknown",
  "org.postgresql.test.jdbc42.GetObject310InfinityTests.test[binary = REGULAR, expr = infinity, pgType = timestamp, klass = class java.time.OffsetDateTime]": "unknown",
}

While looking at this I also realized that we weren't counting failures correctly. JUnit has a concept of tests that "error" in addition to ones that "fail." We were only counting the failed ones as failures. The correct count is 2421 failures out of 5786 total tests.

So in my fix I'll make sure we count everything.

@cockroach-teamcity
Copy link
Member Author

SHA: https://github.com/cockroachdb/cockroach/commits/51a6fdedf0ce1d1329d40d801a7deaf8206b6b07

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stress instead of stressrace and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
stdbuf -oL -eL \
make stressrace TESTS=pgjdbc PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-maxtime 20m -timeout 10m' 2>&1 | tee /tmp/stress.log

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=1428934&tab=buildLog

The test failed on branch=provisional_201908060405_v19.1.4, cloud=gce:
test artifacts and logs in: /home/agent/work/.go/src/github.com/cockroachdb/cockroach/artifacts/20190807-1428934/pgjdbc/run_1
	pgjdbc.go:294,pgjdbc.go:305,test_runner.go:691: 
		Tests run on Cockroach v19.1.3-31-g51a6fde
		Tests run against pgjdbc REL42.2.6
		5786 Total Tests Run
		5768 tests passed
		18 tests failed
		0 tests passed unexpectedly
		4 tests failed unexpectedly
		0 tests expected failed, but not run
		For a full summary look at the pgjdbc artifacts 
		
		An updated blacklist (pgjdbcBlackList19_1) is available in the artifacts' pgjdbc log

@cockroach-teamcity
Copy link
Member Author

SHA: https://github.com/cockroachdb/cockroach/commits/1051f376924f18c5782887ed824ab5ede5027493

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stress instead of stressrace and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
stdbuf -oL -eL \
make stressrace TESTS=pgjdbc PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-maxtime 20m -timeout 10m' 2>&1 | tee /tmp/stress.log

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=1429376&tab=buildLog

The test failed on branch=release-2.1, cloud=gce:
test artifacts and logs in: /home/agent/work/.go/src/github.com/cockroachdb/cockroach/artifacts/20190808-1429376/pgjdbc/run_1
	pgjdbc.go:294,pgjdbc.go:305,test_runner.go:691: 
		Tests run on Cockroach v2.1.8-6-g1051f37
		Tests run against pgjdbc REL42.2.6
		5786 Total Tests Run
		5783 tests passed
		3 tests failed
		12 tests passed unexpectedly
		1 test failed unexpectedly
		0 tests expected failed, but not run
		For a full summary look at the pgjdbc artifacts 
		
		An updated blacklist (pgjdbcBlackList2_1) is available in the artifacts' pgjdbc log

craig bot pushed a commit that referenced this issue Aug 8, 2019
39375: exec: add null handling to default sorter r=rohany a=rohany

Addresses part of #36880.

Release note: None

39420: roachtest: fix pgjdbc tests r=rafiss a=rafiss

The pgjdbc tests have two types of failures: one referred to as
"failures" and the other one as "errors." We were only counting the
former before. Now there are many more test failures that should be
tracked in the blacklist.

touches #39406 

Release note: None

39430: workload/ycsb: implement read-modify-write r=jeffrey-xiao a=jeffrey-xiao

Release note: None

39461: internal/sqlsmith: add vectorize option r=mjibson a=mjibson

This option attempts to limits query generation such that vectorized
execution will occur. Still a lot of work to do but this is a good start.

Release note: None

Co-authored-by: Rohan Yadav <[email protected]>
Co-authored-by: Rafi Shamim <[email protected]>
Co-authored-by: Jeffrey Xiao <[email protected]>
Co-authored-by: Matt Jibson <[email protected]>
@kenliu kenliu mentioned this issue Aug 9, 2019
18 tasks
@cockroach-teamcity
Copy link
Member Author

SHA: https://github.com/cockroachdb/cockroach/commits/51a6fdedf0ce1d1329d40d801a7deaf8206b6b07

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stress instead of stressrace and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
stdbuf -oL -eL \
make stressrace TESTS=pgjdbc PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-maxtime 20m -timeout 10m' 2>&1 | tee /tmp/stress.log

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=1436116&tab=buildLog

The test failed on branch=provisional_201908060405_v19.1.4, cloud=gce:
test artifacts and logs in: /home/agent/work/.go/src/github.com/cockroachdb/cockroach/artifacts/20190812-1436116/pgjdbc/run_1
	pgjdbc.go:306,pgjdbc.go:317,test_runner.go:691: 
		Tests run on Cockroach v19.1.3-31-g51a6fde
		Tests run against pgjdbc REL42.2.6
		5786 Total Tests Run
		3361 tests passed
		2425 tests failed
		0 tests passed unexpectedly
		4 tests failed unexpectedly
		0 tests expected failed, but not run
		For a full summary look at the pgjdbc artifacts 
		
		An updated blacklist (pgjdbcBlackList19_1) is available in the artifacts' pgjdbc log

@cockroach-teamcity
Copy link
Member Author

SHA: https://github.com/cockroachdb/cockroach/commits/179f29b066c266d14cfeac33ce29b2d18ba86c63

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stress instead of stressrace and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
stdbuf -oL -eL \
make stressrace TESTS=pgjdbc PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-maxtime 20m -timeout 10m' 2>&1 | tee /tmp/stress.log

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=1465495&tab=buildLog

The test failed on branch=release-2.1, cloud=gce:
test artifacts and logs in: /home/agent/work/.go/src/github.com/cockroachdb/cockroach/artifacts/20190904-1465495/pgjdbc/run_1
	pgjdbc.go:306,pgjdbc.go:317,test_runner.go:688: 
		Tests run on Cockroach v2.1.8-8-g179f29b
		Tests run against pgjdbc REL42.2.6
		5786 Total Tests Run
		3360 tests passed
		2426 tests failed
		0 tests passed unexpectedly
		2412 tests failed unexpectedly
		0 tests expected failed, but not run
		For a full summary look at the pgjdbc artifacts 
		
		An updated blacklist (pgjdbcBlackList2_1) is available in the artifacts' pgjdbc log

@jordanlewis
Copy link
Member

Something went pretty wrong here - 2412 unexpected failures. @rafiss do you know what happened?

@rafiss
Copy link
Collaborator

rafiss commented Sep 4, 2019

Yes, this is the first time that this suite was run against a 2.1 cluster. I will update the blacklist for 2.1.

@cockroach-teamcity
Copy link
Member Author

SHA: https://github.com/cockroachdb/cockroach/commits/179f29b066c266d14cfeac33ce29b2d18ba86c63

Parameters:

To repro, try:

# Don't forget to check out a clean suitable branch and experiment with the
# stress invocation until the desired results present themselves. For example,
# using stress instead of stressrace and passing the '-p' stressflag which
# controls concurrency.
./scripts/gceworker.sh start && ./scripts/gceworker.sh mosh
cd ~/go/src/github.com/cockroachdb/cockroach && \
stdbuf -oL -eL \
make stressrace TESTS=pgjdbc PKG=roachtest TESTTIMEOUT=5m STRESSFLAGS='-maxtime 20m -timeout 10m' 2>&1 | tee /tmp/stress.log

Failed test: https://teamcity.cockroachdb.com/viewLog.html?buildId=1468506&tab=buildLog

The test failed on branch=provisional_201909042143_v2.1.9, cloud=gce:
test artifacts and logs in: /home/agent/work/.go/src/github.com/cockroachdb/cockroach/artifacts/20190905-1468506/pgjdbc/run_1
	pgjdbc.go:306,pgjdbc.go:317,test_runner.go:688: 
		Tests run on Cockroach v2.1.8-8-g179f29b
		Tests run against pgjdbc REL42.2.6
		5786 Total Tests Run
		3360 tests passed
		2426 tests failed
		0 tests passed unexpectedly
		2412 tests failed unexpectedly
		0 tests expected failed, but not run
		For a full summary look at the pgjdbc artifacts 
		
		An updated blacklist (pgjdbcBlackList2_1) is available in the artifacts' pgjdbc log

craig bot pushed a commit that referenced this issue Sep 5, 2019
40335: coltypes: don't shallow copy decimals r=jordanlewis a=jordanlewis

This commit fixes a correctness bug caused by illegal shallow copy of
decimals. To copy a decimal, you must use the .Set method on it, and its
memory must already be distinct from the source decimal.

To facilitate this and generally simplify things, the "safe" GET method
is removed and replaced by COPYVAL, which copies a scalar value to
another scalar value safely. There were fewer than 5 users of GET, so
this wasn't particularly disruptive.

Closes #39777.
Closes #39540.

Release note: None

40481: roachtest: update pgjdbc blacklist for 2.1 r=rafiss a=rafiss

The 2.1 blacklist was never filled in.

closes #39406

Release note: None

40492: Docs: Add SHOW JOBS diagram r=lhirata a=lhirata

Release note: none

Co-authored-by: Rohan Yadav <[email protected]>
Co-authored-by: Jordan Lewis <[email protected]>
Co-authored-by: Rafi Shamim <[email protected]>
Co-authored-by: Lauren <[email protected]>
@craig craig bot closed this as completed in 3bb9277 Sep 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants