Skip to content

chore(pd-store): cherry-pick commits from master for synchronization & some modifications for passing partial hstore tests#2414

Merged
VGalaxies merged 34 commits intoapache:pd-storefrom
VGalaxies:pd-store-sync
Jan 15, 2024
Merged

chore(pd-store): cherry-pick commits from master for synchronization & some modifications for passing partial hstore tests#2414
VGalaxies merged 34 commits intoapache:pd-storefrom
VGalaxies:pd-store-sync

Conversation

@VGalaxies
Copy link
Copy Markdown
Contributor

Purpose of the PR

Subtask of #2265.

Similar to #2345, also upgrading the dependencies of hugegraph-common to version 1.2.0 within pd and store modules.

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

imbajin and others added 27 commits January 13, 2024 00:41
* 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
* chore: merge master to clean-rocksdb for synchronization (apache#2383)

---------

Co-authored-by: V_Galaxy <[email protected]>
* feat(WIP): add IntMapByDynamicHash (apache#2294)

* feat: add values & keys in IntMapByDynamicHash

* add some basic comment & fix some style

* feat: fix pr review

* fix: fix some review

---------

Co-authored-by: imbajin <[email protected]>
* fix(dist): update relative path for the 1.2.0

* add docker policy and run server in daemon

* fix: use tail to avoid container exit

* adjuest the order of quick start

* Update hugegraph-server/hugegraph-dist/docker/scripts/detect-storage.groovy

* adjust desc

* Update README.md

* Update pom.xml

---------

Co-authored-by: imbajin <[email protected]>
* fix: typo

* fix: assembly hugegraph path

* reset

---------

Co-authored-by: liuxiao <[email protected]>
Co-authored-by: Simon Cheung <[email protected]>
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. ci-cd Build or deploy feature New feature labels Jan 13, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (pd-store@dd44f70). Click here to learn what that means.

❗ Current head 2c84ac0 differs from pull request most recent head b670169. Consider uploading reports for the commit b670169 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             pd-store    #2414   +/-   ##
===========================================
  Coverage            ?   49.58%           
  Complexity          ?     1211           
===========================================
  Files               ?      718           
  Lines               ?    58283           
  Branches            ?     7484           
===========================================
  Hits                ?    28901           
  Misses              ?    26495           
  Partials            ?     2887           

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

@VGalaxies VGalaxies changed the title chore: cherry-pick commits from master for synchronization & upgrade hugegraph-common to 1.2.0 chore(pd-store): cherry-pick commits from master for synchronization & upgrade hugegraph-common to 1.2.0 Jan 13, 2024
@VGalaxies
Copy link
Copy Markdown
Contributor Author

VGalaxies commented Jan 13, 2024

NOTE: We should strive to pass hstore tests before merging, @Pengzna could you please help with this?

@VGalaxies VGalaxies changed the title chore(pd-store): cherry-pick commits from master for synchronization & upgrade hugegraph-common to 1.2.0 chore(pd-store): cherry-pick commits from master for synchronization & some modifications for passing hstore tests Jan 14, 2024
@VGalaxies VGalaxies changed the title chore(pd-store): cherry-pick commits from master for synchronization & some modifications for passing hstore tests chore(pd-store): cherry-pick commits from master for synchronization & some modifications for passing partial hstore tests Jan 15, 2024
@VGalaxies VGalaxies merged commit e11dc8e into apache:pd-store Jan 15, 2024
@VGalaxies VGalaxies deleted the pd-store-sync branch January 15, 2024 17:21
@VGalaxies
Copy link
Copy Markdown
Contributor Author

@Pengzna In next PR, trying to fix potential issues to pass tests in hstore (excluding OLAP scenarios)

@VGalaxies VGalaxies restored the pd-store-sync branch January 15, 2024 17:30
@VGalaxies VGalaxies deleted the pd-store-sync branch January 15, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-cd Build or deploy feature New feature size:XXL This PR changes 1000+ lines, ignoring generated files. tests Add or improve test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.