Skip to content

Try to improve backward compatibility check#38717

Merged
Avogar merged 10 commits intoClickHouse:masterfrom
Avogar:improve-bc-check
Jul 7, 2022
Merged

Try to improve backward compatibility check#38717
Avogar merged 10 commits intoClickHouse:masterfrom
Avogar:improve-bc-check

Conversation

@Avogar
Copy link
Copy Markdown
Member

@Avogar Avogar commented Jul 1, 2022

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Use tests from previous release for backward compatibility check

@Avogar Avogar force-pushed the improve-bc-check branch from 0591993 to 17da24b Compare July 4, 2022 13:02
@Avogar
Copy link
Copy Markdown
Member Author

Avogar commented Jul 5, 2022

@Mergifyio update

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 5, 2022

update

✅ Branch has been successfully updated

@Avogar
Copy link
Copy Markdown
Member Author

Avogar commented Jul 6, 2022

@Mergifyio update

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 6, 2022

update

✅ Branch has been successfully updated

@Avogar
Copy link
Copy Markdown
Member Author

Avogar commented Jul 7, 2022

@Mergifyio update

@Avogar Avogar marked this pull request as ready for review July 7, 2022 09:38
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 7, 2022

update

✅ Branch has been successfully updated

@Avogar
Copy link
Copy Markdown
Member Author

Avogar commented Jul 7, 2022

Next step after this PR - try to remove some no-backward-incompatibility-check:<version> tags from tests

Copy link
Copy Markdown
Member

@tavplubix tavplubix left a comment

Choose a reason for hiding this comment

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

Cool, cloning repo with old tests is even better than filtering current tests!

azat added a commit to azat/ClickHouse that referenced this pull request Feb 15, 2023
This reverts commit 24d5be1, reversing
changes made to a756b4b.

After ClickHouse#38717 there is no need in --no-stress, since there will be no
such checks for previous releases that fails.

Reverts: ClickHouse#38212
Requires: ClickHouse#38717
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants