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

Connection properties file #12

Open
chiquitinxx opened this issue Mar 7, 2014 · 1 comment
Open

Connection properties file #12

chiquitinxx opened this issue Mar 7, 2014 · 1 comment

Comments

@chiquitinxx
Copy link

Will be awesome if new Gstorm() can detect if a file with database connetion properties in the classpath and use it. For example gstorm.properties with something like:

database=myDatabase
user=myUser
password=myPassword
....

@kdabir
Copy link
Owner

kdabir commented Mar 11, 2014

Given that Gstorm's primary use case is scripts, it makes sense to load such file from PATH rather than classpath. IMHO, reading conf from either place is source of confusion for lot of project so making it a part of core gstorm may not add a lot of value.

Moreover it would be easier for applications to load the properties from their desired location and pass required properties on to gstorm instead of gstorm attempting to read file directly.

I am keeping this issue open and I will try to spike out some cases and let other users share their opinions on the issue.

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

No branches or pull requests

2 participants