-
Notifications
You must be signed in to change notification settings - Fork 621
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
Implement CREATE/DROP EXTERNAL DATA SOURCE/EXTERNAL TABLE in QueryService. Without IF NOT EXISTS/IF EXISTS support yet #524
Implement CREATE/DROP EXTERNAL DATA SOURCE/EXTERNAL TABLE in QueryService. Without IF NOT EXISTS/IF EXISTS support yet #524
Conversation
Note This is an automated comment that will be appended during run. 🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit eedc64d.
🔴 linux-x86_64-release-asan: some tests FAILED for commit eedc64d.
|
6d5a6a3
to
fbe2369
Compare
974cf34
to
d9847d6
Compare
ydb/core/kqp/gateway/behaviour/tablestore | ||
ydb/core/kqp/gateway/behaviour/table | ||
ydb/core/kqp/gateway/behaviour/external_data_source | ||
ydb/core/kqp/gateway/utils |
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.
2 раза добавил:)
return MakeFuture(result); | ||
} else { | ||
return Gateway->DropExternalTable(cluster, settings); | ||
} | ||
FORWARD_ENSURE_NO_PREPARE(DropExternalTable, cluster, settings); |
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.
Это больше не нужно
d9847d6
to
2bb1f9c
Compare
…vice. Without IF NOT EXISTS/IF EXISTS support yet
2bb1f9c
to
eedc64d
Compare
Implement CREATE/DROP EXTERNAL DATA SOURCE/EXTERNAL TABLE in QueryService. Without IF NOT EXISTS/IF EXISTS support yet.
Also a common mechanism for working with objects is implemented through
IClassBehaviour::TFactory
. It can be used for all types of objects in SQL