Skip to content

Add tests from #15889#19007

Merged
alexey-milovidov merged 9 commits intomasterfrom
add-tests-from-15889
Jan 16, 2021
Merged

Add tests from #15889#19007
alexey-milovidov merged 9 commits intomasterfrom
add-tests-from-15889

Conversation

@KochetovNicolai
Copy link
Copy Markdown
Member

@KochetovNicolai KochetovNicolai commented Jan 13, 2021

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

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Closes #15889

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jan 13, 2021
@alexey-milovidov alexey-milovidov self-assigned this Jan 13, 2021
@KochetovNicolai
Copy link
Copy Markdown
Member Author

KochetovNicolai commented Jan 13, 2021

Ok, it fails for debug build.

@azat azat mentioned this pull request Jan 15, 2021
Merged
@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Jan 15, 2021

@KochetovNicolai Arcadia does not support distributed queries:

Code: 279. DB::Exception: Received from localhost:35650. DB::Exception: All connection tries failed. Log: 
E   
E   Code: 210, e.displayText() = DB::NetException: Connection refused (localhost:59000) (version 21.1.1.5646-arcadia)
E   Code: 210, e.displayText() = DB::NetException: Connection refused (localhost:59000) (version 21.1.1.5646-arcadia)
E   Code: 210, e.displayText() = DB::NetException: Connection refused (localhost:59000) (version 21.1.1.5646-arcadia)

@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Jan 15, 2021

Table system.query_log doesn't exist

Need one more change to make sure query_log exists.

@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Jan 15, 2021

Ok, I've added SYSTEM FLUSH LOGS.

@alexey-milovidov alexey-milovidov merged commit 32ceafc into master Jan 16, 2021
@alexey-milovidov alexey-milovidov deleted the add-tests-from-15889 branch January 16, 2021 08:03
@KochetovNicolai KochetovNicolai mentioned this pull request Jan 22, 2021
Merged
@CurtizJ CurtizJ mentioned this pull request Feb 13, 2021
Closed
@KochetovNicolai KochetovNicolai mentioned this pull request Feb 16, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexey-milovidov alexey-milovidov alexey-milovidov approved these changes

Assignees

@alexey-milovidov alexey-milovidov

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Block structure mismatch in PipelineExecuting stream

3 participants

@KochetovNicolai @alexey-milovidov @robot-clickhouse