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

Allow control over if we load settings from System Properties when using TransportClient. #2312

Closed
wants to merge 1 commit into from

Conversation

NickPadilla
Copy link

I had problems when trying to use the TransportClient inside a java application plugin - that now uses an internal ES node. They are setting all the ES settings via the System Properties, so my settings get overridden. I am now just allowing a bit more control over how the settings get loaded.

@NickPadilla
Copy link
Author

Here is a link to some more discussion over this topic in the google group :

https://groups.google.com/forum/?fromgroups=#!topic/elasticsearch/jRxu0ZzaKo8

@NickPadilla
Copy link
Author

Hello All - any idea when this fix might make it in a release? Just a general timeline would be helpful. I need to figure out how/when to move this fix into production. It will either mean building a custom jar with the latest fully released codebase or waiting for this fix to make it in. Thanks for the great product!

@NickPadilla
Copy link
Author

@kimchy Hey Shay, any way I could get a timeline on when we could expect this "fix" to be in a release? I don't want o give my client a custom jar and then they update it and it breaks their stuff because this isn't in it. Please advise. Thanks!

@NickPadilla
Copy link
Author

@imotov Hey Igor, is there any way to get an update as to how long this update might take to make it in trunk - or a release? I was hoping it could be in trunk before December, but that is looking unlikely. If there is anything you need from me please let me know.

@NickPadilla
Copy link
Author

After testing the latest release of ES, I have found that the setting provided does work as expected. However, it looks like the property is

config.ignore_system_properties

Thanks a million guys!
Nick

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.

1 participant