In memory coordination inside ClickHouse#19580
Conversation
|
Todo: simplify startup |
done |
|
|
Not related to changes |
|
No related failures, going to merge today or tomorrow. |
cc @zhang2014 |
|
commit 57c9b6c belongs to this pull request excludes darwin(macOS) from running ClickHouse with it is intended? If so, is there a plan to fix? |
Yes, it's intended. I think we will fix it after linux version will be completely ready. |
|
@alesapin where does the original incompatibility come from? NuRaft itself is compatible with OSX. |
@alesapin locally replace That said, |
Yes, but it's not the only problem -- in TCP server handler I use The current development stage is still a prototype, but I can fix OS_DARWIN build if you need it. |
|
@alesapin I'll appreciate it if you fix that. Either fix the type issue, the epoll issue and possible others, or disable NuKeeper module for OS_DARWIN for now. |
But it's already disabled? https://github.com/ClickHouse/ClickHouse/blob/master/src/CMakeLists.txt#L197-L199 |
|
ClickHouse/programs/server/Server.cpp Lines 872 to 874 in 36c178c You're right. I should use the term "run", normally running on OS_DARWIN. However, if it is an significant burden over prototyping, it's ok we give another pass later for the fix. |
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Changelog category (leave one):