Skip to content

[0.17.0-incubating] Fix equalsAndHashCode in ClientCompactQueryTuningConfig (#9035)#9051

Merged
jon-wei merged 1 commit intoapache:0.17.0-incubatingfrom
suneet-s:backport/0.17.0-incubating/pr-9035
Dec 17, 2019
Merged

[0.17.0-incubating] Fix equalsAndHashCode in ClientCompactQueryTuningConfig (#9035)#9051
jon-wei merged 1 commit intoapache:0.17.0-incubatingfrom
suneet-s:backport/0.17.0-incubating/pr-9035

Conversation

@suneet-s
Copy link
Copy Markdown
Contributor

Backports the following commits to 0.17.0-incubating:

@suneet-s suneet-s force-pushed the backport/0.17.0-incubating/pr-9035 branch from 3fb27ec to ea7c167 Compare December 16, 2019 22:37
* Fix equalsAndHashCode in ClientCompactQueryTuningConfig

This change introduces a dependency to EqualsVerifier for the test scope.
The dependency is licensed under Apache 2. The library makes it trivial
to add equals and hashCode checks to prevent bugs like this from happening
in the future

* fix checkstyle

* fix test name
@suneet-s suneet-s force-pushed the backport/0.17.0-incubating/pr-9035 branch from ea7c167 to 3752285 Compare December 16, 2019 23:54
@jon-wei jon-wei merged commit f956a52 into apache:0.17.0-incubating Dec 17, 2019
suneet-s added a commit to implydata/druid-public that referenced this pull request Dec 17, 2019
…apache#9051)

* Fix equalsAndHashCode in ClientCompactQueryTuningConfig

This change introduces a dependency to EqualsVerifier for the test scope.
The dependency is licensed under Apache 2. The library makes it trivial
to add equals and hashCode checks to prevent bugs like this from happening
in the future

* fix checkstyle

* fix test name
@jon-wei jon-wei added this to the 0.17.0 milestone Dec 17, 2019
suneet-s added a commit to implydata/druid-public that referenced this pull request Dec 18, 2019
…apache#9051) (#22)

This change introduces a dependency to EqualsVerifier for the test scope.
The dependency is licensed under Apache 2. The library makes it trivial
to add equals and hashCode checks to prevent bugs like this from happening
in the future
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.

2 participants