Skip to content

feat: catch error msgs in ns/tablet client #8723

feat: catch error msgs in ns/tablet client

feat: catch error msgs in ns/tablet client #8723

Triggered via pull request November 10, 2023 07:25
Status Failure
Total duration 18m 2s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

style.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 11 warnings
cpplint
reviewdog: Too many results (annotations) in diff. You may miss some annotations due to GitHub limitation for annotation created by logging command. Please check GitHub Actions log console to see all results. Limitation: - 10 warning annotations and 10 error annotations per step - 50 annotations per job (sum of annotations from all the steps) - 50 annotations per run (separate from the job annotations, these annotations aren't created by users) Source: https://github.community/t5/GitHub-Actions/Maximum-number-of-annotations-that-can-be-created-using-GitHub/m-p/39085
pylint
Process completed with exit code 2.
cpplint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
cpplint: src/client/ns_client.cc#L478
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/client/ns_client.cc:478: Lines should be <= 120 characters long [whitespace/line_length] [2]
cpplint: src/client/ns_client.cc#L587
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/client/ns_client.cc:587: Lines should be <= 120 characters long [whitespace/line_length] [2]
cpplint: src/client/ns_client.cc#L597
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/client/ns_client.cc:597: Lines should be <= 120 characters long [whitespace/line_length] [2]
cpplint: src/client/tablet_client.cc#L686
[cpplint] reported by reviewdog 🐶 Missing space before ( in if( [whitespace/parens] [5] Raw Output: src/client/tablet_client.cc:686: Missing space before ( in if( [whitespace/parens] [5]
cpplint: src/client/tablet_client.cc#L689
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/client/tablet_client.cc:689: Lines should be <= 120 characters long [whitespace/line_length] [2]
cpplint: src/client/tablet_client.h#L140
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/client/tablet_client.h:140: Lines should be <= 120 characters long [whitespace/line_length] [2]
cpplint: src/cmd/openmldb.cc#L2697
[cpplint] reported by reviewdog 🐶 Missing space after ; [whitespace/semicolon] [3] Raw Output: src/cmd/openmldb.cc:2697: Missing space after ; [whitespace/semicolon] [3]
cpplint: src/cmd/openmldb.cc#L3110
[cpplint] reported by reviewdog 🐶 Lines should be <= 120 characters long [whitespace/line_length] [2] Raw Output: src/cmd/openmldb.cc:3110: Lines should be <= 120 characters long [whitespace/line_length] [2]
cpplint: src/cmd/openmldb.cc#L3227
[cpplint] reported by reviewdog 🐶 Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2] Raw Output: src/cmd/openmldb.cc:3227: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
cpplint: src/cmd/openmldb.cc#L3227
[cpplint] reported by reviewdog 🐶 TODO(my_username) should be followed by a space [whitespace/todo] [2] Raw Output: src/cmd/openmldb.cc:3227: TODO(my_username) should be followed by a space [whitespace/todo] [2]