[GitHub] zookeeper issue #473: ZOOKEEPER-2936 - Removed duplicate code
Github user afine commented on the issue: https://github.com/apache/zookeeper/pull/473 Thanks @asutosh936! ---
[GitHub] zookeeper issue #473: ZOOKEEPER-2936 - Removed duplicate code
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/473 @afine please take a look, I think this is ready to commit. thanks. ---
[GitHub] zookeeper issue #473: ZOOKEEPER-2936 - Removed duplicate code
Github user asutosh936 commented on the issue: https://github.com/apache/zookeeper/pull/473 @anmolnar - Could You or someone from Your team please merge this PR? ---
[GitHub] zookeeper issue #473: ZOOKEEPER-2936 - Removed duplicate code
Github user asutosh936 commented on the issue: https://github.com/apache/zookeeper/pull/473 @anmolnar - Thanks for the Review. The issue is about having duplicate keys for ROLLINGFILE layout and conversion pattern. The fix is to remove the duplicate keys from the file. As rightly said there is no reason of commenting the code, instead deleting it. Please review. ---
[GitHub] zookeeper issue #473: ZOOKEEPER-2936 - Removed duplicate code
Github user asutosh936 commented on the issue: https://github.com/apache/zookeeper/pull/473 Can someone please review and merge this PR. ---