Skip to content
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.

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

Enable some settings #33653

Merged
merged 24 commits into from
Jun 19, 2022
Merged

Enable some settings #33653

merged 24 commits into from
Jun 19, 2022

Conversation

alexey-milovidov
Copy link
Member

@alexey-milovidov alexey-milovidov commented Jan 14, 2022

Changelog category (leave one):

  • Performance optimization and Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Enabled pread_threadpool read method by default. It will increase read performance. Bug fix: if direct IO is enabled and the number of threads is large and pread_threadpool is used, it may cause a logical error.

@robot-clickhouse robot-clickhouse added the pr-backward-incompatible Pull request with backwards incompatible changes label Jan 14, 2022
@alexey-milovidov
Copy link
Member Author

#35596
#35598
#35600

@alexey-milovidov
Copy link
Member Author

Merging with master after #36337.

@alexey-milovidov
Copy link
Member Author

Waiting for #36361

@alexey-milovidov
Copy link
Member Author

It is almost ready, but query_thread_log became too heavy.

@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented May 14, 2022

update

✅ Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@alexey-milovidov
Copy link
Member Author

@mergify
Copy link
Contributor

mergify bot commented May 15, 2022

update

✅ Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

https://github.com/Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jun 16, 2022

update

✅ Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jun 16, 2022

update

☑️ Nothing to do

  • -closed [:pushpin: update requirement]
  • #commits-behind>0 [:pushpin: update requirement]

@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jun 19, 2022

update

✅ Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

Wow, it is fixed!

@alexey-milovidov alexey-milovidov merged commit c941d0c into master Jun 19, 2022
@alexey-milovidov alexey-milovidov deleted the enable-some-settings branch June 19, 2022 16:52
@alexey-milovidov alexey-milovidov added pr-performance Pull request with some performance improvements and removed pr-backward-incompatible Pull request with backwards incompatible changes labels Jun 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-performance Pull request with some performance improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants