Skip to content

Unimplement PartialOrd for TDigest's Centroid#17440

Merged
crepererum merged 1 commit into
apache:mainfrom
findepi:findepi/unimplement-partialord-for-tdigest-s-centroid-66ce0d
Sep 5, 2025
Merged

Unimplement PartialOrd for TDigest's Centroid#17440
crepererum merged 1 commit into
apache:mainfrom
findepi:findepi/unimplement-partialord-for-tdigest-s-centroid-66ce0d

Conversation

@findepi

@findepi findepi commented Sep 5, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

The implementation was not consistent with PartialEq, violating the PartialOrd contract.
@findepi
Unimplement PartialOrd for TDigest's Centroid
ebc09a6 
The implementation was not consistent with PartialEq, violating the
PartialOrd contract.
@github-actions github-actions Bot added the physical-plan Changes to the physical-plan crate label Sep 5, 2025
@findepi findepi mentioned this pull request Sep 5, 2025
Merged
@findepi findepi changed the title Unimplement PartialOrd for TDigest's Centroid Unimplement PartialOrd for TDigest's Centroid Sep 5, 2025
crepererum
crepererum approved these changes Sep 5, 2025
View reviewed changes

@crepererum crepererum left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks
@crepererum
crepererum merged commit df5a09c into apache:main Sep 5, 2025
30 checks passed
@findepi
findepi deleted the findepi/unimplement-partialord-for-tdigest-s-centroid-66ce0d branch September 5, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

physical-plan Changes to the physical-plan crate

2 participants

@findepi @crepererum