GitHub user arunmahadevan opened a pull request:
https://github.com/apache/storm/pull/2811
STORM-3184: Replace the usage of redact-value with ConfigUtils.maskPasswords
The topology submission can fail since redact-value expects a clojure map.
We don't need redact-value, it can be replaced with ConfigUtils.maskPasswords.
This is already done in master.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/arunmahadevan/storm STORM-3184-followup
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2811.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2811
----
commit 43faecc870468a00d5f1b2d5ba1c5c4274ae77b0
Author: Arun Mahadevan <arunm@...>
Date: 2018-08-20T23:31:32Z
STORM-3184: Replace the usage of redact-value with ConfigUtils.maskPasswords
The topology submission can fail since redact-value
expects a clojure map. We dont need redact-value, it can be replaced
with just ConfigUtils.maskPasswords. This is already done in master.
----
---
