-
Notifications
You must be signed in to change notification settings - Fork 638
[YQ-1997] Support for ReplaceIfExists flag in SchemeShard for External Data Source and External Table #1431
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
[YQ-1997] Support for ReplaceIfExists flag in SchemeShard for External Data Source and External Table #1431
Conversation
⚪
|
⚪
|
⚪ |
⚪ |
ydb/core/tx/schemeshard/schemeshard__operation_create_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_create_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_create_external_table.cpp
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_create_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/ut_external_data_source/ut_external_data_source.cpp
Outdated
Show resolved
Hide resolved
UNIT_ASSERT_VALUES_EQUAL(externalDataSourceDescription.GetVersion(), 2); | ||
UNIT_ASSERT_VALUES_EQUAL(externalDataSourceDescription.GetLocation(), "https://s3.cloud.net/my_new_bucket"); | ||
UNIT_ASSERT_EQUAL(externalDataSourceDescription.GetAuth().identity_case(), NKikimrSchemeOp::TAuth::kNone); | ||
} |
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.
We also need test that checks failure when object with the same name but another type already exists.
And also test that checks failure when we have external data source, but ReplaceIfExists flag is not set in command
UNIT_ASSERT_VALUES_EQUAL(columns.Get(1).GetName(), "value"); | ||
UNIT_ASSERT_VALUES_EQUAL(columns.Get(1).GetType(), "Uint64"); | ||
UNIT_ASSERT_VALUES_EQUAL(columns.Get(1).GetNotNull(), false); | ||
} |
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.
We also need test that checks failure when object with the same name but another type already exists.
And also test that checks failure when we have external table, but ReplaceIfExists flag is not set in command
ydb/core/tx/schemeshard/ut_external_table/ut_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/ut_external_table/ut_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_data_source.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_data_source.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_table.cpp
Outdated
Show resolved
Hide resolved
ydb/core/tx/schemeshard/schemeshard__operation_replace_external_table.cpp
Outdated
Show resolved
Hide resolved
Co-authored-by: Vasily Gerasimov <UgnineSirdis@ydb.tech>
Co-authored-by: Vasily Gerasimov <UgnineSirdis@ydb.tech>
⚪
|
⚪
|
Changelog entry
...
Changelog category
Additional information
...