Skip to content

Correct and rework requires interface. Remove unused code.#4

Closed
stub42 wants to merge 2 commits intobcsaller:masterfrom
stub42:stub
Closed

Correct and rework requires interface. Remove unused code.#4
stub42 wants to merge 2 commits intobcsaller:masterfrom
stub42:stub

Conversation

@stub42
Copy link

@stub42 stub42 commented Jun 29, 2016

Backwards incompatible changes, so hopefully not too many people are trying to use the current implementation. The old interface was very limited and only useful for simple charms needing a single non-replicated data source. Real systems need replication and multiple data silos for scale out.

@stub42
Copy link
Author

stub42 commented Aug 1, 2016

This supersedes #3

@stub42
Copy link
Author

stub42 commented Aug 1, 2016

FWIW, charms using the current interface are not suitable for production deploys as they fail as soon as a second PostgreSQL is added, so no redundancy or simple scale out. It also doesn't support multiple data silos (a charm with multiple pgsql relations), nor sharding (where you have 1..n PostgreSQL services related)

@bcsaller
Copy link
Owner

bcsaller commented Aug 2, 2016

This was only ever an example. I suggest we move the repo to one you own,
are you ok with that?

On Mon, Aug 1, 2016 at 2:59 AM stub42 notifications@github.com wrote:

FWIW, charms using the current interface are not suitable for production
deploys as they fail as soon as a second PostgreSQL is added, so no
redundancy or simple scale out. It also doesn't support multiple data silos
(a charm with multiple pgsql relations), nor sharding (where you have 1..n
PostgreSQL services related)


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGVhTSd3XIe4iiiKfaE8mudSSspaiZIks5qbcN_gaJpZM4JBgyh
.

@stub42
Copy link
Author

stub42 commented Aug 2, 2016

Sure. I'll put it in a new home and update the interfaces registry.

@stub42 stub42 closed this Aug 2, 2016
@stub42 stub42 deleted the stub branch August 15, 2016 09:01
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.

2 participants