Skip to content

[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

Merged
merged 12 commits into from
Jan 31, 2024

Conversation

Alnen
Copy link
Contributor

@Alnen Alnen commented Jan 30, 2024

Changelog entry

...

Changelog category

  • New feature

Additional information

...

Copy link

github-actions bot commented Jan 30, 2024

2024-01-30 16:33:54 UTC Pre-commit check for 0459184 has started.
2024-01-30 16:33:56 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-30 17:11:36 UTC Build successful.
2024-01-30 17:11:45 UTC Tests are running...
🔴 2024-01-30 18:49:28 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
16033 15847 0 49 105 32

Copy link

github-actions bot commented Jan 30, 2024

2024-01-30 16:34:09 UTC Pre-commit check for 0459184 has started.
2024-01-30 16:34:11 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-30 17:09:48 UTC Build successful.
2024-01-30 17:10:02 UTC Tests are running...
🔴 2024-01-30 18:40:46 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
60298 50929 0 30 9311 28

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 09:14:48 UTC Pre-commit check for 16db29a has started.
2024-01-31 09:14:49 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-31 09:54:05 UTC Build successful.
2024-01-31 09:54:16 UTC Tests are running...
🔴 2024-01-31 11:08:27 UTC Test run completed, no test results found for commit 73b8d3c. Please check build logs.
2024-01-31 11:08:31 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 09:18:08 UTC Pre-commit check for 16db29a has started.
2024-01-31 09:18:10 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-31 09:54:17 UTC Build successful.
2024-01-31 09:54:29 UTC Tests are running...
🔴 2024-01-31 11:08:23 UTC Test run completed, no test results found for commit 73b8d3c. Please check build logs.
2024-01-31 11:08:26 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 11:09:08 UTC Pre-commit check for 6f62003 has started.
2024-01-31 11:09:10 UTC Build linux-x86_64-release-asan is running...
2024-01-31 11:12:28 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 11:09:45 UTC Pre-commit check for 6f62003 has started.
2024-01-31 11:09:47 UTC Build linux-x86_64-relwithdebinfo is running...
2024-01-31 11:12:29 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 11:12:57 UTC Pre-commit check for 30e65ce has started.
2024-01-31 11:12:58 UTC Build linux-x86_64-release-asan is running...
2024-01-31 11:44:43 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 11:13:40 UTC Pre-commit check for 30e65ce has started.
2024-01-31 11:13:41 UTC Build linux-x86_64-relwithdebinfo is running...
2024-01-31 11:44:44 UTC Check cancelled

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

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

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

Alnen and others added 3 commits January 31, 2024 14:44
Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 11:51:55 UTC Pre-commit check for f768c52 has started.
2024-01-31 11:51:58 UTC Build linux-x86_64-relwithdebinfo is running...
🔴 2024-01-31 12:00:28 UTC Build failed. see the build logs.
2024-01-31 12:01:23 UTC Tests are running...
🔴 2024-01-31 12:22:55 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
51462 41945 0 266 9239 12

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 11:55:08 UTC Pre-commit check for f768c52 has started.
2024-01-31 11:55:10 UTC Build linux-x86_64-release-asan is running...
🔴 2024-01-31 12:05:27 UTC Build failed. see the build logs.
2024-01-31 12:05:37 UTC Tests are running...
🔴 2024-01-31 12:21:13 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
7507 7232 0 261 3 11

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 12:48:57 UTC Pre-commit check for 9ae4db2 has started.
2024-01-31 12:48:59 UTC Build linux-x86_64-relwithdebinfo is running...
2024-01-31 12:53:55 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 12:48:58 UTC Pre-commit check for 9ae4db2 has started.
2024-01-31 12:49:01 UTC Build linux-x86_64-release-asan is running...
2024-01-31 12:53:55 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 13:16:26 UTC Pre-commit check for a708e5f has started.
2024-01-31 13:16:29 UTC Build linux-x86_64-relwithdebinfo is running...
2024-01-31 13:23:34 UTC Check cancelled

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 13:18:55 UTC Pre-commit check for a708e5f has started.
2024-01-31 13:18:57 UTC Build linux-x86_64-release-asan is running...
2024-01-31 13:23:34 UTC Check cancelled

@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
@vitstn vitstn mentioned this pull request Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants