GitHub user thormanrd opened a pull request:
https://github.com/apache/accumulo/pull/32
Accumulo 3652 Refactor for slf4j string substitution. Rebased from master,
style and merge corrections
You can merge this pull request into a Git repository by running:
$ git pull
Github user thormanrd closed the pull request at:
https://github.com/apache/accumulo/pull/31
---
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
Sure, how do I trim them out? They are the result of the 'git pull
--rebase¹ command. When I tried to create the PR without some of them
earlier (PR¹s #30, #31) I got a compiler error from Jenkins.
v/r
Bob Thorman
Principal Big Data Engineer
ATT Big Data CoE
2900 W. Plano Parkway
Plano, TX
Looks like the version of checkstyle used on Accumulo 1.7 doesn't support
Java 7? Are we sure that the job is running in a Java 7 JRE?
-- Forwarded message --
From: Apache Jenkins Server jenk...@builds.apache.org
Date: Thu, Apr 23, 2015 at 11:52 AM
Subject: Accumulo-1.7 - Build #
The job was running with JDK6 by accident. A build #2 should be running.
Mike Drob wrote:
Looks like the version of checkstyle used on Accumulo 1.7 doesn't support
Java 7? Are we sure that the job is running in a Java 7 JRE?
-- Forwarded message --
From: Apache Jenkins
Yup, that was it. Thanks for making sure this got visibility too, Mike.
Josh Elser wrote:
The job was running with JDK6 by accident. A build #2 should be running.
Mike Drob wrote:
Looks like the version of checkstyle used on Accumulo 1.7 doesn't support
Java 7? Are we sure that the job is
Github user joshelser commented on the pull request:
https://github.com/apache/accumulo/pull/32#issuecomment-95655661
Thanks for continuing to work on this, @thormanrd. There still seem to be a
bunch of unwanted changes in here.
https://github.com/thormanrd/accumulo-3652/commit/861494f8ec0a677d5fdf5fbe979399137f895dbc
includes some changes to the class used in a static method call.
https://github.com/thormanrd/accumulo-3652/commit/6d632273f70ca59f9e671d2256dcae5682ee94fd
seems to contain the majority of the rest.
A