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: update pgjdbc blocklists #57081

Merged
merged 1 commit into from
Nov 25, 2020

Conversation

solongordon
Copy link
Contributor

I made the following updates to the pgjdbc blocklists:

  • Removed tests which are now passing due to user-defined schema
    support.
  • Removed testUpdateSelectOnly since we now support this syntax as a
    no-op.
  • Updated some failure reasons to "unknown" for tests which are still
    failing even though the referenced issue was closed.
  • Added many BatchExecuteTest.* tests to the ignore list. These tests
    are flaky due to a combination of sql: cannot perform schema change in transaction after write #54477 and the fact that the tests
    do not run in a deterministic order.

Fixes #53467
Fixes #53738
Fixes #54106

Release note: None

@solongordon solongordon requested review from rafiss and a team November 24, 2020 20:33
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Collaborator

@rafiss rafiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! just a small note about removing the 19.1 stuff

also, just FYI, i believe this will need to backported to each release branch once it is merged. #51897 changed which roachtest binary is used in nightlies

"org.postgresql.test.core.LogServerMessagePropertyTest.testWithExplicitlyEnabled": "expected fail - checks error message",
"org.postgresql.test.core.LogServerMessagePropertyTest.testWithLogServerErrorDetailDisabled": "expected fail - checks error message",
}
var pgjdbcIgnoreList19_2 = pgjdbcIgnoreList19_1

var pgjdbcIgnoreList19_1 = blocklist{
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can remove 19.1 here and just keep the blocklist for 19.2 and up

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. I got rid of all the <19.2 lists.

I made the following updates to the pgjdbc blocklists:
- Removed tests which are now passing due to user-defined schema
  support.
- Removed testUpdateSelectOnly since we now support this syntax as a
  no-op.
- Updated some failure reasons to "unknown" for tests which are still
  failing even though the referenced issue was closed.
- Added many BatchExecuteTest.* tests to the ignore list. These tests
  are flaky due to a combination of cockroachdb#54477 and the fact that the tests
  do not run in a deterministic order.
- Removed lists for versions earlier than v19.2.

Fixes cockroachdb#53467
Fixes cockroachdb#53738
Fixes cockroachdb#54106

Release note: None
@solongordon
Copy link
Contributor Author

Thanks for the heads up about backports!

@solongordon solongordon requested a review from rafiss November 24, 2020 22:05
@solongordon
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Nov 25, 2020

Build succeeded:

@craig craig bot merged commit 8d25fa1 into cockroachdb:master Nov 25, 2020
@solongordon solongordon deleted the fix-pgjdbc-tests branch January 13, 2021 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

roachtest: pgjdbc failed roachtest: pgjdbc failed roachtest: pgjdbc failed
3 participants