GitHub user thormanrd opened a pull request:
https://github.com/apache/accumulo/pull/30
Accumulo 3652 Refactor for slf4j
Worked through all the packages except Shell to ensure slf4j is used and
string concatenation is replace with slf4j substitution.
There were many many files changed and the merge process was very
extensive. All unit test passed as well as mvn clean verify...
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/thormanrd/accumulo-3652 ACCUMULO-3652
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/accumulo/pull/30.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 #30
----
commit eadaf575a802043482585032b66221982675f6df
Author: Bob Thorman <[email protected]>
Date: 2015-04-09T17:23:35Z
ACCUMULO-3652 refactor for slf4j all packages except gc, master, monitor
and server-base
commit fe5e6f80333b1b62ca1883b6854546f756ceac8f
Author: Bob Thorman <[email protected]>
Date: 2015-04-09T18:48:06Z
ACCUMULO-3652 refactor for slf4j for all packages except server-base and
test
commit 3dd645f21b0caf98637a67c9596fce89a9e49d14
Author: Bob Thorman <[email protected]>
Date: 2015-04-09T20:21:22Z
ACCUMULO-3652 refactor for slf4j for every package except accumulo-test
commit 2be6f07e5f6015fffab3c3654e5c2f209088dc17
Author: Bob Thorman <[email protected]>
Date: 2015-04-10T15:40:01Z
ACCUMULO-3652 refactored for slf4j for all packages except Shell
commit 5e8b89d8e6d89a1d85bb2856349d072187fcfcfc
Author: Bob Thorman <[email protected]>
Date: 2015-04-14T15:22:40Z
First round of revisions from the review board.
commit 946ee522d845a093254a60e2c77670e7e9217395
Author: Bob Thorman <[email protected]>
Date: 2015-04-16T15:59:34Z
ACCUMULO-3652 fixed style errors and merge conflicts
commit 6f62012aa075982b6d0b6094fe53b70b0a5950d5
Author: Bob Thorman <[email protected]>
Date: 2015-04-21T15:54:16Z
ACCUMULO-3652 Whitespace corrections, merge conflicts resolution
----
---
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 enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---