Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: async remove left index shouldn't effect query #2199

Merged
merged 1 commit into from
Apr 19, 2023
Merged

Conversation

zyxxoo
Copy link
Contributor

@zyxxoo zyxxoo commented Apr 17, 2023

fix #2081 #2201

@codecov
Copy link

codecov bot commented Apr 19, 2023

Codecov Report

Merging #2199 (9ea28ed) into master (7b6a4bd) will increase coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff            @@
##             master    #2199   +/-   ##
=========================================
  Coverage     65.08%   65.08%           
- Complexity      977      979    +2     
=========================================
  Files           497      497           
  Lines         40566    40572    +6     
  Branches       5663     5663           
=========================================
+ Hits          26403    26407    +4     
- Misses        11548    11551    +3     
+ Partials       2615     2614    -1     
Impacted Files Coverage Δ
.../apache/hugegraph/backend/tx/GraphTransaction.java 79.68% <50.00%> (-0.28%) ⬇️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment on lines +38 to +51
<dependency>
<groupId>io.grpc</groupId>
<artifactId>grpc-netty-shaded</artifactId>
<version>1.47.0</version>
</dependency>

<dependency>
<groupId>org.apache.hugegraph</groupId>
<artifactId>hugegraph-rpc</artifactId>
<exclusions>
<exclusion>
<groupId>io.grpc</groupId>
<artifactId>grpc-netty-shaded</artifactId>
</exclusion>
Copy link
Member

Choose a reason for hiding this comment

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

@z7658329 commons could refer it?

Copy link
Member

Choose a reason for hiding this comment

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

@z7658329 commons could refer it?
ok

@imbajin imbajin changed the title chore: async remove left index should't effect query chore: async remove left index shouldn't effect query Apr 19, 2023
@imbajin imbajin merged commit 267ff6d into master Apr 19, 2023
@imbajin imbajin deleted the zy_dev branch April 19, 2023 13:19
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