Skip to content

Conversation

alesapin
Copy link
Member

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)

Fixes #22080

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 17, 2021
@alesapin
Copy link
Member Author

alesapin commented May 17, 2021

Not 100% sure, I hope we have tests for it.

@alesapin
Copy link
Member Author

alesapin commented May 17, 2021

2021-05-17 21:19:52 00163_column_oriented_formats:                                          Received exception from server (version 21.6.1):
2021-05-17 21:19:52 Code: 999. DB::Exception: Received from localhost:9000. DB::Exception: Connection loss, path: /clickhouse/task_queue/ddl/query-0000000002/finished. 

Why this error is not retried.... We have such message in retry list.

@alesapin
Copy link
Member Author

alesapin commented May 17, 2021

Again nothing for 7 seconds

2021.05.17 21:19:44.643065 [ 608 ] {} <Trace> RaftInstance: local log idx 155, target_commit_idx 155, quick_commit_index_ 155, state_->get_commit_idx() 155
2021.05.17 21:19:45.642596 [ 604 ] {} <Debug> RaftInstance: heartbeat timeout for 3
2021.05.17 21:19:52.909313 [ 604 ] {} <Trace> RaftInstance: (update) new target priority: 3
2021.05.17 21:19:52.909332 [ 604 ] {} <Trace> RaftInstance: send request to 3
2021.05.17 21:19:52.909356 [ 604 ] {} <Trace> RaftInstance: last_log_idx: 155, starting_idx: 1, cur_nxt_idx: 156
2021.05.17 21:19:52.909370 [ 604 ] {} <Debug> RaftInstance: append_entries for 3 with LastLogIndex=155, LastLogTerm=1, EntriesLength=0, CommitIndex=155, Term=1, peer_last_sent_idx 156
2021.05.17 21:19:52.909379 [ 604 ] {} <Trace> RaftInstance: EMPTY PAYLOAD
2021.05.17 21:19:52.909389 [ 604 ] {} <Trace> RaftInstance: send req 1 -> 3, type append_entries_request
2021.05.17 21:19:52.909464 [ 604 ] {} <Trace> RaftInstance: sent

Again absolutely dead disk on test machine :(

@filimonov
Copy link
Contributor

while you are here - what do you think of #23504 ?

@alesapin
Copy link
Member Author

2021-05-17 21:19:52 00163_column_oriented_formats:                                          Received exception from server (version 21.6.1):
2021-05-17 21:19:52 Code: 999. DB::Exception: Received from localhost:9000. DB::Exception: Connection loss, path: /clickhouse/task_queue/ddl/query-0000000002/finished. 

Why this error is not retried.... We have such message in retry list.

that's why #24249

@alesapin
Copy link
Member Author

while you are here - what do you think of #23504 ?

I don't mind it, but not in this PR

@alesapin alesapin merged commit 90f12a3 into master May 19, 2021
@alesapin alesapin deleted the simplier_is_local_address branch May 19, 2021 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-not-for-changelog This PR should not be mentioned in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Client alert: [DB::Exception: I/O error: 19]
3 participants