-
Notifications
You must be signed in to change notification settings - Fork 606
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
hive metastore client has been added #4593
Conversation
⚪ |
⚪ |
⚪ |
✅ Documentation buildRevision built successfully |
bfcfe09
to
69c1062
Compare
⚪
|
⚪
|
что за файлы в gen-cpp? как их получать? можно ли руками редактировать? |
LIBRARY() | ||
|
||
SRCS( | ||
# thrift -r --gen cpp hive_metastore.thrift |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
положи это лучше в readme и укажи точную строчку для генерации, такая явно не взлетит. и должно быть понятно в какой директории запускать.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
добавил readme
gen-cpp/hive_metastore_types.cpp | ||
|
||
# hive metastore external data source | ||
hive_metastore_client.cpp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
давай сторонний код положим в одну либу, а наш - в другую
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Сделать еще одну поддиректорию hive_metastore_native и туда запихнуть gen-cpp share *.thrift и прописать ее как LIBRARY ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
да, так норм
⚪
|
⚪
|
⚪
|
⚪
|
⚪
|
⚪
|
ydb/core/external_sources/hive_metastore/hive_metastore_client.cpp
Outdated
Show resolved
Hide resolved
ydb/core/external_sources/hive_metastore/hive_metastore_client.cpp
Outdated
Show resolved
Hide resolved
TThreadPool ThreadPool; | ||
}; | ||
|
||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
просят newline
cf54814
to
ea2c989
Compare
⚪
|
⚪
|
|
||
void WaitHiveMetastore(const TString& host, int32_t port, const TString& database); | ||
|
||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В нескольких файлах просят пустую строку добавить в конце
⚪
|
⚪
|
Changelog entry
hive metastore client has been added
Changelog category
Additional information
...