Skip to content

feat: Sharded replication pool #2

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.

Sign up for GitHub

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

Jump to bottom
Draft
gtamasi wants to merge 752 commits into
base: main
Choose a base branch
from
Draft

feat: Sharded replication pool #2

gtamasi wants to merge 752 commits into from

Conversation

@gtamasi
Copy link
Collaborator

@gtamasi gtamasi commented Oct 3, 2023

Pull Request Checklist

Please make sure to review and check all of these items:

  • Have you added new tests to prevent regressions?
  • Does yarn test or yarn test-DIALECT pass with this change (including linting)?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
  • Did you update the typescript typings accordingly (if applicable)?
  • Does the description below contain a link to an existing issue (Closes #[issue]) or a description of the issue you are solving?
  • Did you follow the commit message conventions explained in CONTRIBUTING.md?

Description Of Change

Todos

@gtamasi gtamasi self-assigned this Oct 3, 2023
@gtamasi gtamasi changed the title Feature/shard manager feat: Shareded replication pool Oct 9, 2023
@gtamasi gtamasi changed the title feat: Shareded replication pool feat: Sharded replication pool Oct 9, 2023
github-actions[bot] and others added 27 commits April 11, 2024 17:31
v7.0.0-alpha.40
23091a3
@lohart13
fix: make BaseSqlExpression a unique class (sequelize#17158)
6a5ea6c
@renovate @ephys
meta: update dependency @ephys/eslint-config-typescript to v20.1.3 (s…
98db428 
…equelize#17278)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Alyx <[email protected]>
@renovate
meta: update dependency inquirer to v9.2.18
e96f806
@renovate
meta: update dependency oclif to v4.8.2
2d84ae8
@ephys
meta: fix auto-update failing on forks (sequelize#17280)
823baa4
@renovate
meta: update dependency @oclif/core to v3.26.3
14279c8
@renovate
meta: update dependency @oclif/plugin-help to v6.0.21
46b11ac
@renovate
meta: update dependency oclif to v4.8.3
da713d2
@renovate
meta: update dependency typedoc-plugin-mdn-links to v3.1.21
4448f6a
@renovate
meta: update dependency oclif to v4.8.4
94ea02e
@renovate
meta: update dependency @oclif/test to v3.2.10 (sequelize#17284)
ba0b994 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
meta: update dependency @types/snowflake-sdk to v1.6.21
889aa2c
@renovate
meta: update dependency oclif to v4.8.5
fbe4370
@lohart13
fix: set sequelize dialect type in query generator and interface (seq…
0227288 
…uelize#17285)
@renovate
meta: update dependency inquirer to v9.2.19
3d26713
@renovate
meta: update dependency nx to v18.3.0
85cced5
@renovate
meta: update dependency mysql2 to v3.9.5
9a38afd
@renovate
meta: update dependency nx to v18.3.1
9da3489
@renovate
meta: update dependency mysql2 to v3.9.6
78a7760
@renovate
meta: update actions/download-artifact action to v4.1.5
e2f7e57
@renovate
meta: update actions/upload-artifact action to v4.3.2
02b65d9
@renovate
meta: update dependency @oclif/core to v3.26.4
fbae976
@renovate
meta: update dependency nx to v18.3.2
1ed71c3
@renovate
meta: update dependency zod to v3.22.5
3cf2355
@renovate
meta: update dependency eslint-plugin-mocha to v10.4.3
75e032f
@renovate
meta: update dependency nx to v18.3.3
cbc44b7
gtamasi and others added 30 commits July 28, 2024 21:20
@gtamasi
ensure pool is defined
f8a35ca
@gtamasi
ensure the pool is defined in connection-manager
8a3db35
@gtamasi
ensure pool is defined in transaction
ba353c4
@gtamasi
add connection-options
b8228f8
@gtamasi
fix
e21cfa3
@gtamasi
pool!
ed3577a
@gtamasi
hack
44a05bc
@gtamasi
fix
3504e71
@gtamasi
add customTypeParser map
97957f2
@gtamasi
add shardId to connectionOPtions
dffa934
@papandreou
Merge remote-tracking branch 'papandreou/feature/shardManager' into s…
3e0e26a 
…equelize-core-papandreou4

* papandreou/feature/shardManager: (58 commits)
  add shardId to connectionOPtions
  add customTypeParser map
  fix
  hack
  pool!
  fix
  add connection-options
  ensure pool is defined in transaction
  ensure the pool is defined in connection-manager
  ensure pool is defined
  base postgres connection-manager
  add sharding to confing
  add sharded-replication-pool
  rework replication-poll
  add base-replicaion-pool
  rm obsolete files
  small changes
  Add test cases to abstract/connection-manager
  Use read when we usemaster is true when we acquire a pool
  fix cls transaction issue
  ...
@papandreou
Change package name to sequelize-core-papandreou
f5d9614
@papandreou
Release 7.0.0-alpha.41-patch1
afa23ee
@gtamasi
revert version change
20d225c
@papandreou
Manually replace workspace:* with what gets published by yarn
3aaf8c2
@papandreou
Release 7.0.0-alpha.41-patch2
89579cc
@papandreou
Change the name of the @sequelize/postgres package to sequelize-postg…
e7b5e60 
…res-papandreou
@papandreou
Fix workspace:* versions in sequelize-postgres-papandreou
3a6c332
@papandreou
Release [email protected]
0ca51fd
@gtamasi
revert removed eslint comments
1a544bf
@papandreou
Releases [email protected] sequelize-po…
adb2788 
…[email protected]
@papandreou
Temporarily comment out the Object.freeze of the replication options …
63d5104 
…to sidestep ShardedReplicationPool error:

TypeError: Cannot add property shardId, object is not extensible
@papandreou
Add ugly hack to work around bug in ShardedReplicationPool
e18ddfe
@papandreou
Releases [email protected] sequelize-po…
16e6f8e 
…[email protected]
@papandreou
Hack: Clone the writeConfig before handing it to ShardedReplicationPo…
4059620 
…ol so that extending it with shardId later is OK
@papandreou
Releases [email protected] sequelize-po…
125c934 
…[email protected]
@papandreou
Fix this.connectionManager.getConnection => this.pool.acquire from th…
e4934ed 
…e feature/shardManager branch
@papandreou
Releases [email protected] sequelize-po…
fa676af 
…[email protected]
@gtamasi
Merge branch 'sequelize-core-papandreou4' into feature/shardManager
3f4a2ab
@gtamasi
fix passed connection object upon create
06c6609
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@gtamasi gtamasi

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@gtamasi @lohart13 @ephys @WikiRik @webdiscus @wd-alejandroescobedogarcia @papandreou