Skip to content

Bugfix/export ConditionP - #1813

Merged
javeme merged 2 commits into
apache:masterfrom
wuchaojing:bugfix/export_ConditionP
Apr 12, 2022
Merged

Bugfix/export ConditionP#1813
javeme merged 2 commits into
apache:masterfrom
wuchaojing:bugfix/export_ConditionP

Conversation

@wuchaojing

@wuchaojing wuchaojing commented Apr 8, 2022

Copy link
Copy Markdown
Contributor

fix #1812

@github-actions

github-actions Bot commented Apr 8, 2022 •
edited
Loading

Copy link
Copy Markdown

CLA Assistant Lite bot Good! All Contributors have signed the CLA.

@CLAassistant

CLAassistant commented Apr 8, 2022

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

wuchaojing seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov

codecov Bot commented Apr 8, 2022 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #1813 (462c521) into master (d778eb1) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1813      +/-   ##
============================================
- Coverage     66.59%   66.58%   -0.01%     
+ Complexity      982      980       -2     
============================================
  Files           446      446              
  Lines         37902    37902              
  Branches       5398     5398              
============================================
- Hits          25239    25236       -3     
- Misses         9959     9961       +2     
- Partials       2704     2705       +1
Impacted Files Coverage Δ
...egraph/backend/store/cassandra/CassandraStore.java 73.21% <0.00%> (-0.86%) ⬇️

Continue to review full report at Codecov.

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

javeme
javeme approved these changes Apr 8, 2022
View reviewed changes

@javeme javeme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your contribution

@wuchaojing

wuchaojing commented Apr 8, 2022

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@imbajin

imbajin commented Apr 11, 2022 •
edited
Loading

Copy link
Copy Markdown
Member

@wuchaojing seems your commit email is not same with github's account, maybe try to update it to make CLA right.

image

github-actions Bot added a commit that referenced this pull request Apr 11, 2022
@github-actions
@javeme has signed the CLA from Pull Request #1813
c2640f8
@javeme
javeme merged commit 0a3599d into apache:master Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@javeme javeme javeme approved these changes
@imbajin imbajin imbajin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] set类型contains查询报错

4 participants

@wuchaojing @CLAassistant @imbajin @javeme