ClickHouse / ClickHouse

ClickHouse® is a real-time analytics DBMS
https://clickhouse.com
Apache License 2.0
34.9k stars 6.55k forks source link

Fix cannot create SEQUENTIAL node with keeper-client #64177

Open canhld94 opened 1 week ago

canhld94 commented 1 week ago

Changelog category (leave one):

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

/ :) create /clickhouse/tmp- 'bar' PERSISTENT SEQUENTIAL
Syntax error: failed at position 41 (' '):

create /clickhouse/tmp- 'bar' PERSISTENT SEQUENTIAL

Expected end of query

Documentation entry for user-facing changes

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

Modify your CI run **NOTE:** If your merge the PR with modified CI you **MUST KNOW** what you are doing **NOTE:** Checked options will be applied if set before CI RunConfig/PrepareRunConfig step #### Include tests (required builds will be added automatically): - [ ] Fast test - [ ] Integration Tests - [ ] Stateless tests - [ ] Stateful tests - [ ] Unit tests - [ ] Performance tests - [ ] All with ASAN - [ ] All with TSAN - [ ] All with Analyzer - [ ] All with Azure - [ ] Add your option here #### Exclude tests: - [ ] Fast test - [ ] Integration Tests - [ ] Stateless tests - [x] Stateful tests - [x] Performance tests - [ ] All with ASAN - [ ] All with TSAN - [ ] All with MSAN - [ ] All with UBSAN - [ ] All with Coverage - [ ] All with Aarch64 - [ ] Add your option here #### Extra options: - [ ] do not test (only style check) - [ ] disable merge-commit (no merge from master before tests) - [ ] disable CI cache (job reuse) #### Only specified batches in multi-batch jobs: - [ ] 1 - [ ] 2 - [ ] 3 - [ ] 4
robot-clickhouse-ci-1 commented 1 week ago

This is an automated comment for commit 37f1d59d122b1b37d7be4fa2386297957255a81c with description of existing statuses. It's updated for the latest CI running

❌ Click here to open a full report in a separate page

Check nameDescriptionStatus
A SyncThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS⏳ pending
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR⏳ pending
ClickHouse build checkBuilds ClickHouse in various configurations for use in further steps. You have to fix the builds that fail. Build logs often has enough information to fix the error, but you might have to reproduce the failure locally. The cmake options can be found in the build log, grepping for cmake. Use these options and follow the general build process❌ failure
Mergeable CheckChecks if all other necessary checks are successful❌ failure
Successful checks
Check nameDescriptionStatus
Docs checkBuilds and tests the documentation✅ success
Fast testNormally this is the first check that is ran for a PR. It builds ClickHouse and runs most of stateless functional tests, omitting some. If it fails, further checks are not started until it is fixed. Look at the report to see which tests fail, then reproduce the failure locally as described here✅ success
Integration testsThe integration tests report. In parenthesis the package type is given, and in square brackets are the optional part/total tests✅ success
PR CheckThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS✅ success
Stateless testsRuns stateless functional tests for ClickHouse binaries built in various configurations -- release, debug, with sanitizers, etc✅ success
Style checkRuns a set of checks to keep the code style clean. If some of tests failed, see the related log from the report✅ success
Unit testsRuns the unit tests for different release types✅ success