Skip to content

Revert "Merge pull request #75735 from yariks5s/time/time64_v2" #80700

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
nikitamikhaylov merged 3 commits into from
May 27, 2025
Merged

Revert "Merge pull request #75735 from yariks5s/time/time64_v2" #80700

nikitamikhaylov merged 3 commits into from
May 27, 2025

Conversation

@nikitamikhaylov
Copy link
Member

@nikitamikhaylov nikitamikhaylov commented May 22, 2025 •
edited
Loading

This reverts commit 654dc82, reversing changes made to f497100.

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

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

This reverts #75735 #80327

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link

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

Workflow [PR], commit [180c764]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 22, 2025
@nikitamikhaylov nikitamikhaylov mentioned this pull request May 22, 2025
Closed
@Algunenano Algunenano self-assigned this May 22, 2025
@yariks5s yariks5s mentioned this pull request May 22, 2025
Closed
1 task
@nikitamikhaylov nikitamikhaylov force-pushed the revert-time branch 2 times, most recently from 77af023 to e6646cd Compare May 23, 2025 11:51
@nikitamikhaylov
Copy link
Member Author

nikitamikhaylov commented May 27, 2025

Performance tests: #78555, there should be no difference detected.

@nikitamikhaylov nikitamikhaylov added this pull request to the merge queue May 27, 2025
Merged via the queue into master with commit 90b0f7e May 27, 2025
118 of 121 checks passed
@nikitamikhaylov nikitamikhaylov deleted the revert-time branch May 27, 2025 01:30
@robot-ch-test-poll3 robot-ch-test-poll3 mentioned this pull request May 27, 2025
Merged
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label May 27, 2025
yariks5s added a commit that referenced this pull request May 28, 2025
@yariks5s
Merge pull request #80864 from ClickHouse/cherrypick/25.5/80700
7457203 
Cherry pick #80700 to 25.5: Revert "Merge pull request #75735 from yariks5s/time/time64_v2"
robot-clickhouse added a commit that referenced this pull request May 28, 2025
@robot-clickhouse
Backport #80700 to 25.5: Revert "Merge pull request #75735 from yarik…
b29953c 
…s5s/time/time64_v2"
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 mentioned this pull request May 28, 2025
Merged
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label May 28, 2025
Algunenano added a commit that referenced this pull request May 29, 2025
@Algunenano
Merge pull request #80966 from ClickHouse/backport/25.5/80700
dd23c80 
Backport #80700 to 25.5: Revert "Merge pull request #75735 from yariks5s/time/time64_v2"
yariks5s added a commit that referenced this pull request Jun 3, 2025
@yariks5s
Revert "Merge pull request #80700 from ClickHouse/revert-time"
efe8280 
This reverts commit 90b0f7e, reversing
changes made to 7a763c1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Algunenano Algunenano Algunenano approved these changes

Assignees

@Algunenano Algunenano

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore 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

@nikitamikhaylov @Algunenano @robot-clickhouse @robot-ch-test-poll1