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

Allow to run multiple instances in parallel #37

Closed
DDani opened this Issue Apr 30, 2013 · 5 comments

Comments

Projects
None yet
2 participants
@DDani

DDani commented Apr 30, 2013

Hi,
currently when dynamic-dynamodb starts in daemon mode, it creates a pid file on this location:
daemon = DynamicDynamoDBDaemon('/tmp/dynamic-dynamodb.pid')

Since this is harcoded, it is not possible to run multiple instances od dynamic-dynamodb.

Regards,

@ghost ghost assigned sebdah Apr 30, 2013

@sebdah

This comment has been minimized.

Owner

sebdah commented Apr 30, 2013

Perfect, thanks. I should have thought about that. Will look into it.

@sebdah

This comment has been minimized.

Owner

sebdah commented Apr 30, 2013

I have been looking into the code slightly. I am thinking about implementing a --instance / --instance-name flag as a command line option. The flag should be set differently to control different daemons. If the flag is optional and if it is not set Dynamic DynamoDB will control the default daemon only.

Will think a bit more about it. But the implementation is quite straight forward.

@DDani

This comment has been minimized.

DDani commented Apr 30, 2013

I like this approach, much better than the one I was adding just know (by adding a --pid-file-name arg).

@sebdah

This comment has been minimized.

Owner

sebdah commented May 1, 2013

This has been implemented and the name of the daemon instances are set using the --instance flag. If the flag is not set it defaults to a default daemon name.

Thanks for the feedback @DDani !

@sebdah sebdah closed this May 1, 2013

@sebdah

This comment has been minimized.

Owner

sebdah commented May 1, 2013

FYI, Dynamic DynamoDB 1.3.0 has been released to PyPI. You can upgrade to the latest version with pip install -U dynamic-dynamodb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment