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

Implement CREATE/DROP EXTERNAL DATA SOURCE/EXTERNAL TABLE in QueryService. Without IF NOT EXISTS/IF EXISTS support yet #524

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

UgnineSirdis
Copy link
Collaborator

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

Copy link

github-actions bot commented Dec 15, 2023

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit eedc64d.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
57862 48704 0 31 9124 3

🔴 linux-x86_64-release-asan: some tests FAILED for commit eedc64d.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14080 13963 0 37 72 8

@UgnineSirdis UgnineSirdis requested review from gridnevvvit and VPolka and removed request for gridnevvvit December 18, 2023 10:33
@UgnineSirdis UgnineSirdis force-pushed the external-table-ddl branch 2 times, most recently from 974cf34 to d9847d6 Compare December 18, 2023 20:31
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
Copy link
Collaborator

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);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это больше не нужно

…vice. Without IF NOT EXISTS/IF EXISTS support yet
@UgnineSirdis UgnineSirdis merged commit bf58f60 into ydb-platform:main Dec 20, 2023
@UgnineSirdis UgnineSirdis deleted the external-table-ddl branch December 20, 2023 05:27
This was referenced Dec 29, 2023
Closed
This was referenced Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants