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

refactor(clustering): change sync data structure #13794

Merged

Conversation

chronolaw
Copy link
Contributor

Summary

KAG-5679

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

@github-actions github-actions bot added core/db/migrations core/clustering cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels Oct 28, 2024
@chronolaw chronolaw changed the title Refactor/change sync data structure refactor(clustering): change sync data structure Oct 28, 2024
@chronolaw chronolaw force-pushed the refactor/tuning_inc_sync branch from 150fa2d to c625a60 Compare October 28, 2024 03:15
@chronolaw chronolaw force-pushed the refactor/change_sync_data_structure branch from 5c293d0 to 3346e41 Compare October 28, 2024 03:22
@chronolaw chronolaw marked this pull request as ready for review October 28, 2024 05:13
@chronolaw chronolaw merged commit b075641 into refactor/tuning_inc_sync Oct 28, 2024
28 checks passed
@chronolaw chronolaw deleted the refactor/change_sync_data_structure branch October 28, 2024 05:13
chronolaw added a commit that referenced this pull request Oct 31, 2024
* rename db id to pk

* change id to pk in Lua

* change pk to json

* schema:extract_pk_values
bungle pushed a commit that referenced this pull request Nov 1, 2024
* fix(db): pagesize must not be less than 2
* fix(db): set global key in lmdb
* fix(db): use request_aware_table correctly
* fix(dbless): get default workspace correctly
* fix(clustering): check latest_version and d.ws_id
* fix(clustering): fix dp do_sync()
* fix(clustering):fix declarative export sync
* fix(clustering):check latest_version == ngx_null in CP
* fix(clustering): inc sync add ws (#13790)
* fix(clustering/sync): set ws_id in do_sync()
* fix(clustering): delta version is null (#13789)
* fix(clustering/sync): delta.version may be null
* refactor(clustering): change sync data structure (#13794)
* fix(incremental sync): added comment for 2 times do_sync (#13801)
* fix(incremental sync): fix cache_key handling for select_by_cache_key (#13815)
* fix(incremental sync): fix cache_key handling for select_by_cache_key
* refactor(dbless): clean logic of select_by_field (#13817)
* refactor(dbless): clean logic of selec_by_field
* fix(incremental sync): fixed ws_id processing in _set_entity_for_txn (#13816)

---------

Co-authored-by: Xiaochen Wang <[email protected]>
Co-authored-by: Datong Sun <[email protected]>
lhanjian pushed a commit that referenced this pull request Dec 23, 2024
* fix(db): pagesize must not be less than 2
* fix(db): set global key in lmdb
* fix(db): use request_aware_table correctly
* fix(dbless): get default workspace correctly
* fix(clustering): check latest_version and d.ws_id
* fix(clustering): fix dp do_sync()
* fix(clustering):fix declarative export sync
* fix(clustering):check latest_version == ngx_null in CP
* fix(clustering): inc sync add ws (#13790)
* fix(clustering/sync): set ws_id in do_sync()
* fix(clustering): delta version is null (#13789)
* fix(clustering/sync): delta.version may be null
* refactor(clustering): change sync data structure (#13794)
* fix(incremental sync): added comment for 2 times do_sync (#13801)
* fix(incremental sync): fix cache_key handling for select_by_cache_key (#13815)
* fix(incremental sync): fix cache_key handling for select_by_cache_key
* refactor(dbless): clean logic of select_by_field (#13817)
* refactor(dbless): clean logic of selec_by_field
* fix(incremental sync): fixed ws_id processing in _set_entity_for_txn (#13816)

---------

Co-authored-by: Xiaochen Wang <[email protected]>
Co-authored-by: Datong Sun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants