Skip to content

Time/Time64 comparison #80327

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.

Sign up for GitHub

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

Jump to bottom
Merged
yakov-olkhovskiy merged 2 commits into from
May 16, 2025
Merged

Time/Time64 comparison #80327

yakov-olkhovskiy merged 2 commits into from
May 16, 2025
+92 −56

Conversation

@yariks5s
Copy link
Member

@yariks5s yariks5s commented May 16, 2025 •
edited
Loading

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Support comparison between Time/Time64

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link

clickhouse-gh bot commented May 16, 2025 •
edited
Loading

Workflow [PR], commit [4753c7c]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 16, 2025
@Algunenano
Copy link
Member

Algunenano commented May 16, 2025

Not for changelog (changelog entry is not required)

This looks like it's a fix for something. Please change the category and link the issue if exists

@yakov-olkhovskiy yakov-olkhovskiy self-assigned this May 16, 2025
@yakov-olkhovskiy yakov-olkhovskiy added this pull request to the merge queue May 16, 2025
Merged via the queue into master with commit f54bad7 May 16, 2025
118 of 121 checks passed
@yakov-olkhovskiy yakov-olkhovskiy deleted the yarik/time-time64-comparison branch May 16, 2025 15:11
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-synced-to-cloud The PR is synced to the cloud repo label May 16, 2025
nikitamikhaylov added a commit that referenced this pull request May 22, 2025
@nikitamikhaylov
Revert "Merge pull request #80327 from ClickHouse/yarik/time-time64-c…
9fd8472 
…omparison"

This reverts commit f54bad7, reversing
changes made to b611eb9.
@nikitamikhaylov nikitamikhaylov mentioned this pull request May 22, 2025
Merged
1 task
nikitamikhaylov added a commit that referenced this pull request May 22, 2025
@nikitamikhaylov
Revert "Merge pull request #80327 from ClickHouse/yarik/time-time64-c…
fb04a50 
…omparison"

This reverts commit f54bad7, reversing
changes made to b611eb9.
nikitamikhaylov added a commit that referenced this pull request May 23, 2025
@nikitamikhaylov
Revert "Merge pull request #80327 from ClickHouse/yarik/time-time64-c…
b663dbd 
…omparison"

This reverts commit f54bad7, reversing
changes made to b611eb9.
nikitamikhaylov added a commit that referenced this pull request May 23, 2025
@nikitamikhaylov
Revert "Merge pull request #80327 from ClickHouse/yarik/time-time64-c…
9f0cb87 
…omparison"

This reverts commit f54bad7, reversing
changes made to b611eb9.
@yariks5s yariks5s mentioned this pull request Jun 3, 2025
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yakov-olkhovskiy yakov-olkhovskiy yakov-olkhovskiy approved these changes

Assignees

@yakov-olkhovskiy yakov-olkhovskiy

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@yariks5s @Algunenano @yakov-olkhovskiy @robot-ch-test-poll3