Skip to content

Add scan/insert while alter test #8094

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 3 commits into from
Aug 22, 2024

Conversation

fexolm
Copy link
Contributor

@fexolm fexolm commented Aug 21, 2024

Changelog entry

...

Changelog category

  • New feature
  • Experimental feature
  • Improvement
  • Performance improvement
  • Bugfix
  • Backward incompatible change
  • Documentation (changelog entry is not required)
  • Not for changelog (changelog entry is not required)

Additional information

...

@fexolm fexolm requested a review from vgvoleg August 21, 2024 10:51
@fexolm fexolm force-pushed the fexolm/olap-stability3 branch 2 times, most recently from 46ce48e to b6e232a Compare August 21, 2024 11:12
Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 11:16:56 UTC Pre-commit check for 557367c has started.
2024-08-21 11:19:18 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-21 11:20:07 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
2 1 0 1 0 0

🟢 2024-08-21 11:20:14 UTC Build successful.

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 11:19:03 UTC Pre-commit check for 557367c has started.
2024-08-21 11:21:21 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-21 11:22:51 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
4 3 0 1 0 0

2024-08-21 11:22:58 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟡 2024-08-21 11:23:15 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1 (only retried tests) 0 0 1 0 0

2024-08-21 11:23:23 UTC Failed tests rerun (try 3) linux-x86_64-relwithdebinfo is running...
🔴 2024-08-21 11:23:39 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1 (only retried tests) 0 0 1 0 0

🟢 2024-08-21 11:23:46 UTC Build successful.

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 11:19:09 UTC Pre-commit check for 557367c has started.
2024-08-21 11:21:46 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-21 11:22:32 UTC Build successful.

@fexolm fexolm force-pushed the fexolm/olap-stability3 branch from b6e232a to a4155da Compare August 21, 2024 11:32
Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 11:33:30 UTC Pre-commit check for a803cdf has started.
2024-08-21 11:35:45 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-21 11:36:20 UTC Build successful.

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 11:35:45 UTC Pre-commit check for a803cdf has started.
2024-08-21 11:38:04 UTC Check linux-x86_64-relwithdebinfo is running...
🟢 2024-08-21 11:38:43 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
4 4 0 0 0 0

🟢 2024-08-21 11:38:50 UTC Build successful.


def select_10(self, table_name):
print(f"Select 10 {table_name}")
self.driver.table_client.scan_query(f"SELECT * FROM {table_name} limit 10").next()
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do you specify so small limit?


data = []

for i in range(10):
Copy link
Collaborator

Choose a reason for hiding this comment

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

make batch size a parameter

print(f"Add column {table_name}.{col_name}")

def callee(session):
session.execute_scheme(f"ALTER TABLE {table_name} ADD COLUMN {col_name} Int64")
Copy link
Collaborator

Choose a reason for hiding this comment

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

add different types, including var len strings

data = []

for i in range(10):
data.append({c: random.randint(0, 1000000) for c in schema})
Copy link
Collaborator

Choose a reason for hiding this comment

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

skip some columns

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 16:41:42 UTC Pre-commit check for bc0bc41 has started.
2024-08-21 16:44:01 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-21 16:44:41 UTC Build successful.

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 16:41:51 UTC Pre-commit check for bc0bc41 has started.
2024-08-21 16:44:05 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-21 16:44:49 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
2 1 0 1 0 0

🟢 2024-08-21 16:44:56 UTC Build successful.

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 16:42:28 UTC Pre-commit check for bc0bc41 has started.
2024-08-21 16:46:19 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-21 16:47:10 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
4 3 0 1 0 0

2024-08-21 16:47:18 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟡 2024-08-21 16:47:36 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1 (only retried tests) 0 0 1 0 0

2024-08-21 16:47:43 UTC Failed tests rerun (try 3) linux-x86_64-relwithdebinfo is running...
🔴 2024-08-21 16:48:01 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1 (only retried tests) 0 0 1 0 0

🟢 2024-08-21 16:48:08 UTC Build successful.

zverevgeny
zverevgeny previously approved these changes Aug 21, 2024
Copy link

github-actions bot commented Aug 22, 2024

2024-08-22 07:44:15 UTC Pre-commit check for 4be39c8 has started.
2024-08-22 07:46:29 UTC Check linux-x86_64-relwithdebinfo is running...
🟢 2024-08-22 07:47:20 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
4 4 0 0 0 0

🟢 2024-08-22 07:47:28 UTC Build successful.

Copy link

github-actions bot commented Aug 22, 2024

2024-08-22 07:44:19 UTC Pre-commit check for 4be39c8 has started.
2024-08-22 07:46:38 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-22 07:47:18 UTC Build successful.

Copy link

github-actions bot commented Aug 22, 2024

2024-08-22 07:44:33 UTC Pre-commit check for 4be39c8 has started.
2024-08-22 07:46:54 UTC Check linux-x86_64-release-asan is running...
🟢 2024-08-22 07:47:36 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
2 2 0 0 0 0

🟢 2024-08-22 07:47:43 UTC Build successful.

@fexolm fexolm enabled auto-merge (squash) August 22, 2024 09:39
@fexolm fexolm merged commit baf6413 into ydb-platform:main Aug 22, 2024
12 checks passed
rekby pushed a commit to rekby-forks/ydb that referenced this pull request Aug 28, 2024
stanislav-shchetinin pushed a commit to stanislav-shchetinin/ydb that referenced this pull request Aug 30, 2024
zverevgeny pushed a commit to zverevgeny/ydb that referenced this pull request Sep 13, 2024
zverevgeny pushed a commit to zverevgeny/ydb that referenced this pull request Sep 14, 2024
zverevgeny pushed a commit to zverevgeny/ydb that referenced this pull request Sep 15, 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