feat (sync service): multi tenancy #1886
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kevin-dp
force-pushed
the
kevin/multi-tenancy-continued
branch
from
023c50d
to
51f005c
Compare
… dev and test included
On CI we don't wait for the database to start accepting connections before running Electric. And we cannot check the response body of the health check request. So adjusting the response status is the only remaining option.
…onment, dev and test included
kevin-dp
force-pushed
the
kevin/multi-tenancy-continued
branch
2 times, most recently
from
b1c6c26
to
f5def2e
Compare
kevin-dp
force-pushed
the
kevin/multi-tenancy-continued
branch
from
f5def2e
to
835a72f
Compare
✅ Deploy Preview for electric-next ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…in multi tenant mode.
kevin-dp
force-pushed
the
kevin/multi-tenancy-continued
branch
from
7e96fdc
to
bc347db
Compare
kevin-dp
force-pushed
the
kevin/multi-tenancy-continued
branch
from
bc347db
to
775190e
Compare
kevin-dp
force-pushed
the
kevin/multi-tenancy-continued
branch
from
9a4bd76
to
2a888bf
Compare
…nit tests running concurrently with multi tenancy integration test.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Supersedes #1868. I kept that PR up to consult its diff until all the issues have been resolved here.
This PR fixes #1591.
It enables Electric to work with multiple databases (i.e. tenants).
Architectural overview
TODO:
database_idquery param to the health endpoint to request the health of a given tenantvalidate_tenant_id,load_tenantandassign_tenantwhich currently are duplicated in the create shape plug and delete shape plug.