Skip to content

Conversation

@zhanglei1949
Copy link
Collaborator

@zhanglei1949 zhanglei1949 commented Sep 23, 2024

  • Refactor the codegen for stored procedure, let adhoc query and procedure query all inherit from AdhocReadApp.
  • Refine the create_procedure and call_procedure test in sdk test.
  • Fix argument parsing problem.

Committed-by: xiaolei.zl from Dev container
@github-actions
Copy link
Contributor

github-actions bot commented Sep 23, 2024

Please check the preview of the documentation changes at
https://dda507bf.graphscope-docs-preview.pages.dev

Committed-by: xiaolei.zl from Dev container
Committed-by: xiaolei.zl from Dev container
Committed-by: xiaolei.zl from Dev container
Committed-by: xiaolei.zl from Dev container
Committed-by: xiaolei.zl from Dev container
@codecov-commenter
Copy link

codecov-commenter commented Oct 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.78%. Comparing base (ae4e9a2) to head (038f64c).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #4246       +/-   ##
===========================================
+ Coverage   36.96%   58.78%   +21.82%     
===========================================
  Files         126       12      -114     
  Lines       13235      592    -12643     
===========================================
- Hits         4892      348     -4544     
+ Misses       8343      244     -8099     

see 114 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae4e9a2...038f64c. Read the comment docs.

Committed-by: xiaolei.zl from Dev container
@zhanglei1949 zhanglei1949 requested a review from lnfjpt October 12, 2024 05:54
@zhanglei1949 zhanglei1949 merged commit 5d89a61 into alibaba:main Oct 12, 2024
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.

4 participants