GitHub user praste reopened a pull request:
https://github.com/apache/zookeeper/pull/148
ZOOKEEPER-2659 Log4j 2 migration
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/praste/zookeeper log4j2-migration
Alternatively you can
Github user praste commented on the issue:
https://github.com/apache/zookeeper/pull/148
Reopening as multiple people are interested in it. However, I have switched
jobs and may not be able to make further changes until I get approval from my
new employer.
---
If your project is
Github user praste closed the pull request at:
https://github.com/apache/zookeeper/pull/148
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user praste commented on the issue:
https://github.com/apache/zookeeper/pull/148
Looks like no one is really interested in this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user praste commented on the issue:
https://github.com/apache/zookeeper/pull/148
can this be merged ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user praste commented on the issue:
https://github.com/apache/zookeeper/pull/148
Two options
1. Use patch
* Use link
https://patch-diff.githubusercontent.com/raw/apache/zookeeper/pull/148.diff and
save it as patch. Then use `git apply-patch`
2. Pull from
Github user praste commented on the issue:
https://github.com/apache/zookeeper/pull/148
Are sure you removed reference to `slf4j-log4j12` from all the `ivy.xml`
files?
I am not an ivy expert but you can take a look at
http://stackoverflow.com/questions/5405310/find-hidden
Github user praste commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/148#discussion_r98013864
--- Diff:
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
@@ -413,13 +418,18 @@ public void testBadPeerAddressInQuorum
Github user praste commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/148#discussion_r97232914
--- Diff:
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
@@ -413,13 +418,18 @@ public void testBadPeerAddressInQuorum
Github user praste commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/148#discussion_r96629041
--- Diff:
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
@@ -413,13 +418,18 @@ public void testBadPeerAddressInQuorum
Github user praste commented on the issue:
https://github.com/apache/zookeeper/pull/148
I have made all the suggested changes except for using List appender,
which I will look into tonight
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user praste commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/148#discussion_r96327023
--- Diff:
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
@@ -413,13 +418,18 @@ public void testBadPeerAddressInQuorum
Github user praste commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/148#discussion_r96277825
--- Diff: conf/log4j2.xml ---
@@ -0,0 +1,56 @@
+
+
+
+
+%d{ISO8601} [myid:%X{myid}] -
%-5p [%t:%C{1}@%L] - %m%n
--- End diff
Github user praste commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/148#discussion_r96277523
--- Diff: ivy.xml ---
@@ -41,13 +41,20
GitHub user praste opened a pull request:
https://github.com/apache/zookeeper/pull/148
ZOOKEEPER-2664 Log4j2 migration
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/praste/zookeeper log4j2-migration
Alternatively you can
15 matches
Mail list logo
