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

Support post-install table creation #14

Open
aabrams opened this issue Jun 9, 2016 · 4 comments
Open

Support post-install table creation #14

aabrams opened this issue Jun 9, 2016 · 4 comments

Comments

@aabrams
Copy link

aabrams commented Jun 9, 2016

In our app, table creation is done per module at module init phase, separate from db connection init.

Could an API be made for that?

@thejsj
Copy link
Owner

thejsj commented Jun 12, 2016

Let me see if I understand the use case correctly:

  • You have one app connecting to rethinkdb
  • Instead of creating tables in one place, you create them in multiple places depending on module responsibility and scope
  • You want to create tables in each module using the same connection you use everywhere else

Does that sound about right? If not, could you explain the use case more clearly and maybe provide a more detailed explanation of what the API would look like.

Happy to take a look.

@aabrams
Copy link
Author

aabrams commented Jun 12, 2016

What you described sounds exactly right, we share the same connection.

On Sun, Jun 12, 2016, 20:56 Jorge Silva [email protected] wrote:

Let me see if I understand the use case correctly:

  • You have one app connecting to rethinkdb
  • Instead of creating tables in one place, you create them in multiple
    places depending on module responsibility and scope
  • You want to create tables in each module using the same connection
    you use everywhere else

Does that sound about right? If not, could you explain the use case more
clearly and maybe provide a more detailed explanation of what the API would
look like.

Happy to take a look.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#14 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAYBoIlNEKjn0tnRp9tfMxqWLp_u5umvks5qLEg9gaJpZM4Ixv4b
.

@thejsj
Copy link
Owner

thejsj commented Jun 13, 2016

This should be pretty simple, let me see if I can try making that part of the API public in some way.

@aabrams
Copy link
Author

aabrams commented Jun 13, 2016

Thanks!

On Mon, Jun 13, 2016, 20:38 Jorge Silva [email protected] wrote:

This should be pretty simple, let me see if I can try making that part of
the API public in some way.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#14 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAYBoLsfNRNxGvUpralkG-2y9Q1vfEx0ks5qLZWwgaJpZM4Ixv4b
.

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

No branches or pull requests

2 participants