Re: Merge of patch in Flume-3021?

2018-02-13 Thread Mike Percy
OK. In the pull request, it would be nice if whoever submits or merges it mentions all of the contributors to the patch in the commit message. I asked Wolfgang H. about the SolrServer thing and this is what he told me: Hi Mike, the class has been renamed to "SolrClient" (which unfortunately >

Re: Store password for config safely?

2018-02-13 Thread Mike Percy
I think Ferenc has been looking at something related to this, or perhaps is trying to get an existing patch merged (FLUME-2442 , PR 197 ). I haven't been following that work closely so I don't know if it's

[GitHub] flume pull request #197: FLUME-2442 Need an alternative to providing clear t...

2018-02-13 Thread szaboferee
GitHub user szaboferee opened a pull request: https://github.com/apache/flume/pull/197 FLUME-2442 Need an alternative to providing clear text passwords in flume config Adding an interface with 3 implementations to provide functionality at the configuration level to replace

RE: Merge of patch in Flume-3021?

2018-02-13 Thread Wahrmann, Helmut
Hi Mike, Thanks for the response. Would be cool if we get that sorted out. I've asked Yonghao Zou to submit the Pull Request, since he did most of the work and should get the credit. He'll do so after the Chinese New Year's Eve. I will then issue a Pull request for the new ES Rest client,