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: merge master to clean-rocksdb for synchronization #2383

Merged
merged 33 commits into from
Dec 11, 2023

Conversation

VGalaxies
Copy link
Contributor

Purpose of the PR

As title, and resolved some conflicts.

Subtask of #2200.

Main Changes

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows:
    • CI

Does this PR potentially affect the following parts?

  • Nope
  • Dependencies (add/update license info)
  • Modify configurations
  • The public API
  • Other affects (typed here)

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need

msgui and others added 30 commits September 11, 2023 10:48
tips:
- this feat works when auth mode was set.
- this feat works when white ip status was enabled.

because now PD is unavailable,just use java list; when pd ready , we can checkout pd.
## Main Changes

1. change the dockerfile, adding the shell to wait for storage backend and use a docker-entrypoint.sh to manage the starting process.
2. delete a deprecated class in  gremlin-console.sh (reference: [doc of ScriptExecutor](https://tinkerpop.apache.org/javadocs/3.2.3/full/org/apache/tinkerpop/gremlin/groovy/jsr223/ScriptExecutor.html))
3. add a healthy check in docker-compose
4. add an example folder where we can put all the template docker-compose.yml here
5. add `*swagger-ui*` in gitignore, which appears after you compile the source code locally.

---------

Co-authored-by: imbajin <[email protected]>
## Main Changes

- Enhance Consumers.java, supporting ExceptionHandle and `Future` to handle InterruptedException when awaiting
- Add Nested Iterator Edge and support batch execution
- Support batch execution & thread parallel in KoutTraverser and Kneighbor
…apache#2337)

add swagger belong for arthas API

---------

Co-authored-by: imbajin <[email protected]>
* chore(api): code style for cr

---------

Co-authored-by: imbajin <[email protected]>
apache#2338)

## Purpose of the PR

Subtask of apache#2265.

Adjust the project structure of this repository to include three sub-modules: hugegraph-server, hugegraph-pd, hugegraph-store at the root level.

## Main Changes

Roll back to the moment when apache#2266 was merged on `pd-store` and incorporate the latest changes in `master`.

For more detailed information, please refer to apache#2266 (comment).


---------

Co-authored-by: M <[email protected]>
* fix(api): refactor/downgrade record logic for slow log

add some TODOs & assign @SunnyBoy-WYH to address it

* fix typo

* enhance the perf
* feat: dockerfile adapt the project structure

* change the version

---------

Co-authored-by: imbajin <[email protected]>
…2356)

* fix: remove redirect master role to align with behaviour of VertexApi and EdgeApi

* chore: add back necessary annotation
* feat(server):swagger support auth for standardAuth mode and try to fix arthas odd test

* chore(api): update api version & swagger token auth mode
* Update LICENSE

* Update install-cassandra.sh

* add stage profile settings

* Update ci.yml

* Update ci.yml

* Update ci.yml

* Update licence-checker.yml

---------

Co-authored-by: imbajin <[email protected]>
* chore: move server info into GlobalMasterInfo

Change-Id: Id854892333115fd45d7c8b9799d255627541b2ad

* fix testClearAndInit() and export GlobalMasterInfo

Change-Id: Ic2878c5359e7f55fcd11986aa0bf79241c7ee9ab

* enhence ServerInfoManager.heartbeat()

Change-Id: I6932893c4be8331547f3b721083ca00430f85e58

* move RoleElectionStateMachineTest to UnitTestSuite

Change-Id: I7b3e8e2867dcf1063a726c8005f1c34dbd218f7c
…he#2365)

* update common version and fix rest client problem

* use stage

* fix dependency issue

* test

* test

* test

* test

* fix

* chore:  improve the ci logic

* fix core-version

* Empty test

* fix code issue

* fix: 3rd party changes

* code optimize

* refactor the AbsJerseyRestClient

* fix code issue

---------

Co-authored-by: imbajin <[email protected]>
* chore: reset version to 1.2.0

* chore: add README for three submodules

* fix: README.md

* fix: README.md

* fix: README.md

* fix: README.md
Copy link

codecov bot commented Dec 11, 2023

Codecov Report

Attention: 364 lines in your changes are missing coverage. Please review.

Comparison is base (23776a7) 61.56% compared to head (eaa1df8) 66.29%.
Report is 18 commits behind head on clean-rocksdb.

Files Patch % Lines
...g/apache/hugegraph/api/profile/WhiteIpListAPI.java 0.00% 57 Missing ⚠️
...e/hugegraph/api/traversers/CustomizedPathsAPI.java 0.00% 29 Missing ⚠️
...che/hugegraph/api/traversers/SameNeighborsAPI.java 14.70% 29 Missing ⚠️
...a/org/apache/hugegraph/api/traversers/KoutAPI.java 54.00% 14 Missing and 9 partials ⚠️
...a/org/apache/hugegraph/api/metrics/MetricsAPI.java 89.79% 14 Missing and 6 partials ⚠️
.../apache/hugegraph/api/traversers/KneighborAPI.java 54.76% 11 Missing and 8 partials ⚠️
...pi/src/main/java/org/apache/hugegraph/api/API.java 51.72% 13 Missing and 1 partial ⚠️
...che/hugegraph/api/traversers/EdgeExistenceAPI.java 0.00% 12 Missing ⚠️
...rg/apache/hugegraph/serializer/JsonSerializer.java 83.33% 4 Missing and 8 partials ⚠️
...che/hugegraph/api/filter/AuthenticationFilter.java 16.66% 9 Missing and 1 partial ⚠️
... and 25 more
Additional details and impacted files
@@                 Coverage Diff                 @@
##             clean-rocksdb    #2383      +/-   ##
===================================================
+ Coverage            61.56%   66.29%   +4.73%     
+ Complexity             979      828     -151     
===================================================
  Files                  498      510      +12     
  Lines                40682    42133    +1451     
  Branches              5681     5836     +155     
===================================================
+ Hits                 25044    27932    +2888     
+ Misses               13126    11436    -1690     
- Partials              2512     2765     +253     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@imbajin imbajin merged commit 7cec2a7 into apache:clean-rocksdb Dec 11, 2023
20 of 21 checks passed
imbajin added a commit that referenced this pull request Dec 11, 2023
* chore: merge master to clean-rocksdb for synchronization (#2383)

---------

Co-authored-by: V_Galaxy <[email protected]>
VGalaxies added a commit to VGalaxies/incubator-hugegraph that referenced this pull request Jan 12, 2024
* chore: merge master to clean-rocksdb for synchronization (apache#2383)

---------

Co-authored-by: V_Galaxy <[email protected]>
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.