Skip to content

ci: add sqlness job#835

Merged
v0y4g3r merged 7 commits intoGreptimeTeam:developfrom
waynexia:sqlness-ci
Jan 10, 2023
Merged

ci: add sqlness job#835
v0y4g3r merged 7 commits intoGreptimeTeam:developfrom
waynexia:sqlness-ci

Conversation

@waynexia
Copy link
Copy Markdown
Member

@waynexia waynexia commented Jan 5, 2023

Signed-off-by: Ruihang Xia waynestxia@gmail.com

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

run our sqlness tests in CI

Checklist

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 5, 2023

Codecov Report

Merging #835 (6d7d89a) into develop (72f05a3) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #835      +/-   ##
===========================================
- Coverage    86.26%   86.09%   -0.17%     
===========================================
  Files          418      418              
  Lines        54776    54776              
===========================================
- Hits         47253    47161      -92     
- Misses        7523     7615      +92     
Flag Coverage Δ
rust 86.09% <ø> (-0.17%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/sql/src/error.rs 81.96% <ø> (ø)
src/object-store/src/test_util.rs 0.00% <0.00%> (-100.00%) ⬇️
src/datanode/src/instance.rs 52.35% <0.00%> (-17.28%) ⬇️
src/meta-srv/src/handler/collect_stats_handler.rs 81.08% <0.00%> (-13.52%) ⬇️
tests-integration/src/test_util.rs 86.89% <0.00%> (-12.63%) ⬇️
src/servers/src/error.rs 34.54% <0.00%> (-5.46%) ⬇️
src/servers/src/grpc.rs 86.95% <0.00%> (-4.35%) ⬇️
src/servers/src/auth/user_provider.rs 82.71% <0.00%> (-2.47%) ⬇️
src/servers/src/opentsdb.rs 96.82% <0.00%> (-1.59%) ⬇️
src/common/substrait/src/df_logical.rs 77.02% <0.00%> (-0.88%) ⬇️
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Copy link
Copy Markdown
Member

@killme2008 killme2008 left a comment

Choose a reason for hiding this comment

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

LGTM

@v0y4g3r v0y4g3r self-requested a review January 10, 2023 07:12
Copy link
Copy Markdown
Contributor

@v0y4g3r v0y4g3r left a comment

Choose a reason for hiding this comment

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

LGTM

@v0y4g3r v0y4g3r merged commit 1305924 into GreptimeTeam:develop Jan 10, 2023
@waynexia waynexia deleted the sqlness-ci branch January 10, 2023 07:17
paomian pushed a commit to paomian/greptimedb that referenced this pull request Oct 19, 2023
* ci: add sqlness job

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* update sqlness to official release

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* filter out backtrace

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* fix error display

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

* close once_cell feature gate

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants