Skip to content

Integration test for part_log table#7620

Merged
alexey-milovidov merged 6 commits intoClickHouse:masterfrom
millb:integration_test_for_part_log_table
Nov 4, 2019
Merged

Integration test for part_log table#7620
alexey-milovidov merged 6 commits intoClickHouse:masterfrom
millb:integration_test_for_part_log_table

Conversation

@millb
Copy link
Contributor

@millb millb commented Nov 4, 2019

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

For changelog. Remove if this is non-significant change.

Category (leave one):

  • Testing

Short description (up to few sentences):
Relevant problem is #7123. Added integration test for part_log table.
...

Detailed description (optional):

...

node1.query("CREATE TABLE test_table(word String, value UInt64) ENGINE=MergeTree() ORDER BY value")
assert "Table system.part_log doesn't exist" in node1.query_and_get_error("SELECT * FROM system.part_log")
node1.query("INSERT INTO test_table VALUES ('name', 1)")
time.sleep(10)
Copy link
Member

Choose a reason for hiding this comment

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

Ok. But also you can:

  • lower the flush period in config;
  • issue a SYSTEM FLUSH LOGS query.

@alexey-milovidov alexey-milovidov merged commit 3ee7de2 into ClickHouse:master Nov 4, 2019
@CurtizJ CurtizJ added the pr-build Pull request with build/testing/packaging improvement label Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-build Pull request with build/testing/packaging improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants