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

sm allow cassandra backend creating thread as needed while execute cql #896

Merged
merged 2 commits into from
Mar 16, 2020

Conversation

zhoney
Copy link
Contributor

@zhoney zhoney commented Mar 16, 2020

fixed: #892

Change-Id: I586eb64c90c475508247c3f8a39ae487598dd518

https://github.com/hugegraph/hugegraph/pull/892#issuecomment-598545072

fixed: #892

Change-Id: I586eb64c90c475508247c3f8a39ae487598dd518
@zhoney
Copy link
Contributor Author

zhoney commented Mar 16, 2020

没写测试用例,触发条件比较苛刻:需要后端是Cassandra集群,且单个查询涉及多个节点。

@codecov
Copy link

codecov bot commented Mar 16, 2020

Codecov Report

Merging #896 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #896      +/-   ##
============================================
- Coverage     72.23%   72.23%   -0.01%     
+ Complexity     4487     4486       -1     
============================================
  Files           287      287              
  Lines         21411    21412       +1     
  Branches       3017     3017              
============================================
  Hits          15467    15467              
  Misses         4432     4432              
- Partials       1512     1513       +1     
Impacted Files Coverage Δ Complexity Δ
.../baidu/hugegraph/security/HugeSecurityManager.java 84.57% <100.00%> (+0.08%) 82.00 <0.00> (ø)
...c/main/java/com/baidu/hugegraph/task/HugeTask.java 70.93% <0.00%> (-0.39%) 64.00% <0.00%> (-1.00%)

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 9166c26...35c04ad. Read the comment docs.

ImmutableSet.of("open", "opened", "init"),
/*
* fixed https://github
* .com/hugegraph/hugegraph/pull/892#issuecomment-598545072
Copy link
Contributor

Choose a reason for hiding this comment

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

keep on a line.
or move 892#issuecomment-598545072 to this pr and just add comment "fix #896"

Change-Id: I4fbe3af1026f7c9c88c26765a34e4af0ffefd81c
@javeme javeme merged commit b9a7c9b into master Mar 16, 2020
@javeme javeme deleted the sm-cas-thread branch March 16, 2020 11:23
javeme pushed a commit that referenced this pull request Jun 18, 2020
#896)

fixed: #892

Change-Id: I586eb64c90c475508247c3f8a39ae487598dd518
Linary added a commit that referenced this pull request Jun 19, 2020
* Fix mysql backend openWithoutDB ssl-mode not work (#842)
* fix bug of missing offset with index query (#866)
* allow system async task in gremlin context (#892)
* sm allow cassandra backend creating thread as needed while execute cql (#896)
* fix: Invalid limit 10000000, must be <= capacity (#950)
* Fix gremlin-console can't work (#1027)

Change-Id: Ice222d809e4161f1d4d1f04822bef1c5af380a10

Co-authored-by: Linary <liningrui@vip.qq.com>
Co-authored-by: zhoney <zhangyi51@baidu.com>
zhoney added a commit that referenced this pull request Sep 22, 2020
#896)

fixed: #892

Change-Id: I586eb64c90c475508247c3f8a39ae487598dd518
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.

None yet

3 participants