Skip to content

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

@github-actions
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 VPolka and gridnevvvit 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 раза добавил:)

} 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