[1/3] accumulo git commit: ACCUMULO-3838 Fix race condition in ReplicationIT

2015-05-20 Thread elserj
Repository: accumulo
Updated Branches:
  refs/heads/1.7 25d9b2854 -> 81fdad8d1
  refs/heads/master 9cd82738e -> cf9b9a4ea


ACCUMULO-3838 Fix race condition in ReplicationIT

It was possible that the master sees a WAL that's closed,
copies it to replication and then deletes it from metadata
before the test sees the log entry.

Make sure that the test observes wals from metadata or
replication to make sure it passes.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/81fdad8d
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/81fdad8d
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/81fdad8d

Branch: refs/heads/1.7
Commit: 81fdad8d1b09a919feaa15885cd0b95bf15daa6e
Parents: 25d9b28
Author: Josh Elser 
Authored: Wed May 20 23:13:37 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 23:59:59 2015 -0400

--
 .../test/replication/ReplicationIT.java | 144 ++-
 1 file changed, 73 insertions(+), 71 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/81fdad8d/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
--
diff --git 
a/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java 
b/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
index 34c699e..e2ee215 100644
--- a/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
+++ b/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
@@ -41,6 +41,7 @@ import org.apache.accumulo.core.client.IteratorSetting;
 import org.apache.accumulo.core.client.IteratorSetting.Column;
 import org.apache.accumulo.core.client.Scanner;
 import org.apache.accumulo.core.client.TableNotFoundException;
+import org.apache.accumulo.core.client.TableOfflineException;
 import org.apache.accumulo.core.client.ZooKeeperInstance;
 import org.apache.accumulo.core.client.admin.TableOperations;
 import org.apache.accumulo.core.conf.Property;
@@ -135,6 +136,7 @@ public class ReplicationIT extends ConfigurableMacIT {
 scanner.setRange(new Range());
 for (Entry entry : scanner) {
   if (Thread.interrupted()) {
+Thread.currentThread().interrupt();
 return logs;
   }
 
@@ -148,6 +150,31 @@ public class ReplicationIT extends ConfigurableMacIT {
 return logs;
   }
 
+  private Multimap getAllLogs(Connector conn) throws 
TableNotFoundException {
+Multimap logs = getLogs(conn);
+try {
+  Scanner scanner = conn.createScanner(ReplicationTable.NAME, 
Authorizations.EMPTY);
+  StatusSection.limit(scanner);
+  Text buff = new Text();
+  for (Entry entry : scanner) {
+if (Thread.interrupted()) {
+  Thread.currentThread().interrupt();
+  return logs;
+}
+
+StatusSection.getFile(entry.getKey(), buff);
+String file = buff.toString();
+StatusSection.getTableId(entry.getKey(), buff);
+String tableId = buff.toString();
+
+logs.put(file, tableId);
+  }
+} catch (TableOfflineException e) {
+  log.debug("Replication table isn't online yet");
+}
+return logs;
+  }
+
   private void waitForGCLock(Connector conn) throws InterruptedException {
 // Check if the GC process has the lock before wasting our retry attempts
 ZooKeeperInstance zki = (ZooKeeperInstance) conn.getInstance();
@@ -513,7 +540,7 @@ public class ReplicationIT extends ConfigurableMacIT {
 // when that happens
 while (keepRunning.get()) {
   try {
-logs.putAll(getLogs(conn));
+logs.putAll(getAllLogs(conn));
   } catch (TableNotFoundException e) {
 log.error("Metadata table doesn't exist");
   }
@@ -604,6 +631,7 @@ public class ReplicationIT extends ConfigurableMacIT {
 while (observedLogs.hasNext()) {
   Entry observedLog = observedLogs.next();
   if (replicationTableId.equals(observedLog.getValue())) {
+log.info("Removing {} because its tableId is for the replication 
table", observedLog);
 observedLogs.remove();
   }
 }
@@ -612,9 +640,10 @@ public class ReplicationIT extends ConfigurableMacIT {
 // They might not yet all be closed though (might be newfile)
 Assert.assertEquals("Metadata log distribution: " + logs, logs.keySet(), 
replFiles);
 
+final Configuration conf = new Configuration();
 for (String replFile : replFiles) {
   Path p = new Path(replFile);
-  FileSystem fs = p.getFileSystem(new Configuration());
+  FileSystem fs = p.getFileSystem(conf);
   Assert.assertTrue("File does not exist anymore, it was likely 
incorrectly garbage collected: " + p, fs.exists(p));
 }
   }
@@ -665,86 +694,59 @@ public class

[3/3] accumulo git commit: Merge branch '1.7'

2015-05-20 Thread elserj
Merge branch '1.7'

Conflicts:

test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/cf9b9a4e
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/cf9b9a4e
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/cf9b9a4e

Branch: refs/heads/master
Commit: cf9b9a4ea6cfd03a053e64c7feca964ca9c026aa
Parents: 9cd8273 81fdad8
Author: Josh Elser 
Authored: Thu May 21 00:03:56 2015 -0400
Committer: Josh Elser 
Committed: Thu May 21 00:03:56 2015 -0400

--
 .../test/replication/ReplicationIT.java | 93 +++-
 1 file changed, 50 insertions(+), 43 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/cf9b9a4e/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
--
diff --cc 
test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
index 54b42f4,e2ee215..ef81f2c
--- a/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
+++ b/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
@@@ -132,32 -130,21 +133,33 @@@ public class ReplicationIT extends Conf
}
  
private Multimap getLogs(Connector conn) throws 
TableNotFoundException {
 -Multimap logs = HashMultimap.create();
 +// Map of server to tableId
 +Multimap serverToTableID = HashMultimap.create();
  Scanner scanner = conn.createScanner(MetadataTable.NAME, 
Authorizations.EMPTY);
 -scanner.fetchColumnFamily(LogColumnFamily.NAME);
 -scanner.setRange(new Range());
 +scanner.setRange(MetadataSchema.TabletsSection.getRange());
 +
scanner.fetchColumnFamily(MetadataSchema.TabletsSection.CurrentLocationColumnFamily.NAME);
 +for (Entry entry : scanner) {
 +  TServerInstance key = new TServerInstance(entry.getValue(), 
entry.getKey().getColumnQualifier());
 +  byte[] tableId = KeyExtent.tableOfMetadataRow(entry.getKey().getRow());
 +  serverToTableID.put(key, new String(tableId, UTF_8));
 +}
 +// Map of logs to tableId
 +Multimap logs = HashMultimap.create();
 +scanner = conn.createScanner(MetadataTable.NAME, Authorizations.EMPTY);
 +scanner.setRange(MetadataSchema.CurrentLogsSection.getRange());
  for (Entry entry : scanner) {
if (Thread.interrupted()) {
+ Thread.currentThread().interrupt();
  return logs;
}
 -
 -  LogEntry logEntry = LogEntry.fromKeyValue(entry.getKey(), 
entry.getValue());
 -
 -  for (String log : logEntry.logSet) {
 -// Need to normalize the log file from LogEntry
 -logs.put(new Path(log).toString(), 
logEntry.extent.getTableId().toString());
 +  Text path = new Text();
 +  MetadataSchema.CurrentLogsSection.getPath(entry.getKey(), path);
 +  Text session = new Text();
 +  Text hostPort = new Text();
 +  MetadataSchema.CurrentLogsSection.getTabletServer(entry.getKey(), 
hostPort, session);
 +  TServerInstance server = new 
TServerInstance(AddressUtil.parseAddress(hostPort.toString(), false), 
session.toString());
 +  for (String tableId : serverToTableID.get(server)) {
 +logs.put(new Path(path.toString()).toString(), tableId);
}
  }
  return logs;
@@@ -575,12 -638,12 +603,13 @@@
  
  // We should have *some* reference to each log that was seen in the 
metadata table
  // They might not yet all be closed though (might be newfile)
 -Assert.assertEquals("Metadata log distribution: " + logs, logs.keySet(), 
replFiles);
 +Assert.assertTrue("Metadata log distribution: " + logs + "replFiles " + 
replFiles, logs.keySet().containsAll(replFiles));
 +Assert.assertTrue("Difference between replication entries and current 
logs is bigger than one", logs.keySet().size() - replFiles.size() <= 1);
  
+ final Configuration conf = new Configuration();
  for (String replFile : replFiles) {
Path p = new Path(replFile);
-   FileSystem fs = p.getFileSystem(new Configuration());
+   FileSystem fs = p.getFileSystem(conf);
Assert.assertTrue("File does not exist anymore, it was likely 
incorrectly garbage collected: " + p, fs.exists(p));
  }
}
@@@ -631,53 -694,59 +660,31 @@@
  conn.securityOperations().grantTablePermission("root", 
ReplicationTable.NAME, TablePermission.WRITE);
  conn.tableOperations().deleteRows(ReplicationTable.NAME, null, null);
  
- final AtomicBoolean keepRunning = new AtomicBoolean(true);
- final Set metadataWals = new HashSet<>();
- 
- Thread t = new Thread(new Runnable() {
-   @Override
-   public void run() {
- // Should really be able to interrupt here, but the Scanner throws a 
fit to the logger

[2/3] accumulo git commit: ACCUMULO-3838 Fix race condition in ReplicationIT

2015-05-20 Thread elserj
ACCUMULO-3838 Fix race condition in ReplicationIT

It was possible that the master sees a WAL that's closed,
copies it to replication and then deletes it from metadata
before the test sees the log entry.

Make sure that the test observes wals from metadata or
replication to make sure it passes.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/81fdad8d
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/81fdad8d
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/81fdad8d

Branch: refs/heads/master
Commit: 81fdad8d1b09a919feaa15885cd0b95bf15daa6e
Parents: 25d9b28
Author: Josh Elser 
Authored: Wed May 20 23:13:37 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 23:59:59 2015 -0400

--
 .../test/replication/ReplicationIT.java | 144 ++-
 1 file changed, 73 insertions(+), 71 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/81fdad8d/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
--
diff --git 
a/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java 
b/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
index 34c699e..e2ee215 100644
--- a/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
+++ b/test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
@@ -41,6 +41,7 @@ import org.apache.accumulo.core.client.IteratorSetting;
 import org.apache.accumulo.core.client.IteratorSetting.Column;
 import org.apache.accumulo.core.client.Scanner;
 import org.apache.accumulo.core.client.TableNotFoundException;
+import org.apache.accumulo.core.client.TableOfflineException;
 import org.apache.accumulo.core.client.ZooKeeperInstance;
 import org.apache.accumulo.core.client.admin.TableOperations;
 import org.apache.accumulo.core.conf.Property;
@@ -135,6 +136,7 @@ public class ReplicationIT extends ConfigurableMacIT {
 scanner.setRange(new Range());
 for (Entry entry : scanner) {
   if (Thread.interrupted()) {
+Thread.currentThread().interrupt();
 return logs;
   }
 
@@ -148,6 +150,31 @@ public class ReplicationIT extends ConfigurableMacIT {
 return logs;
   }
 
+  private Multimap getAllLogs(Connector conn) throws 
TableNotFoundException {
+Multimap logs = getLogs(conn);
+try {
+  Scanner scanner = conn.createScanner(ReplicationTable.NAME, 
Authorizations.EMPTY);
+  StatusSection.limit(scanner);
+  Text buff = new Text();
+  for (Entry entry : scanner) {
+if (Thread.interrupted()) {
+  Thread.currentThread().interrupt();
+  return logs;
+}
+
+StatusSection.getFile(entry.getKey(), buff);
+String file = buff.toString();
+StatusSection.getTableId(entry.getKey(), buff);
+String tableId = buff.toString();
+
+logs.put(file, tableId);
+  }
+} catch (TableOfflineException e) {
+  log.debug("Replication table isn't online yet");
+}
+return logs;
+  }
+
   private void waitForGCLock(Connector conn) throws InterruptedException {
 // Check if the GC process has the lock before wasting our retry attempts
 ZooKeeperInstance zki = (ZooKeeperInstance) conn.getInstance();
@@ -513,7 +540,7 @@ public class ReplicationIT extends ConfigurableMacIT {
 // when that happens
 while (keepRunning.get()) {
   try {
-logs.putAll(getLogs(conn));
+logs.putAll(getAllLogs(conn));
   } catch (TableNotFoundException e) {
 log.error("Metadata table doesn't exist");
   }
@@ -604,6 +631,7 @@ public class ReplicationIT extends ConfigurableMacIT {
 while (observedLogs.hasNext()) {
   Entry observedLog = observedLogs.next();
   if (replicationTableId.equals(observedLog.getValue())) {
+log.info("Removing {} because its tableId is for the replication 
table", observedLog);
 observedLogs.remove();
   }
 }
@@ -612,9 +640,10 @@ public class ReplicationIT extends ConfigurableMacIT {
 // They might not yet all be closed though (might be newfile)
 Assert.assertEquals("Metadata log distribution: " + logs, logs.keySet(), 
replFiles);
 
+final Configuration conf = new Configuration();
 for (String replFile : replFiles) {
   Path p = new Path(replFile);
-  FileSystem fs = p.getFileSystem(new Configuration());
+  FileSystem fs = p.getFileSystem(conf);
   Assert.assertTrue("File does not exist anymore, it was likely 
incorrectly garbage collected: " + p, fs.exists(p));
 }
   }
@@ -665,86 +694,59 @@ public class ReplicationIT extends ConfigurableMacIT {
 conn.securityOperations().grantTablePermission("root", 
ReplicationTable.

svn commit: r952136 - in /websites/production/accumulo/content: ./ 1.4/apidocs/ 1.5/apidocs/ 1.6/apidocs/ 1.7/apidocs/

2015-05-20 Thread ctubbsii
Author: ctubbsii
Date: Thu May 21 02:31:11 2015
New Revision: 952136

Log:
Publish site: Add Trent Nadeau to contributors.

Added:
websites/production/accumulo/content/
  - copied from r952135, websites/staging/accumulo/trunk/content/
websites/production/accumulo/content/1.4/apidocs/
  - copied from r952135, websites/production/accumulo/content/1.4/apidocs/
websites/production/accumulo/content/1.5/apidocs/
  - copied from r952135, websites/production/accumulo/content/1.5/apidocs/
websites/production/accumulo/content/1.6/apidocs/
  - copied from r952135, websites/production/accumulo/content/1.6/apidocs/
websites/production/accumulo/content/1.7/apidocs/
  - copied from r952135, websites/production/accumulo/content/1.7/apidocs/



svn commit: r952135 - in /websites/staging/accumulo/trunk/content: ./ people.html

2015-05-20 Thread buildbot
Author: buildbot
Date: Thu May 21 02:29:12 2015
New Revision: 952135

Log:
Staging update by buildbot for accumulo

Modified:
websites/staging/accumulo/trunk/content/   (props changed)
websites/staging/accumulo/trunk/content/people.html

Propchange: websites/staging/accumulo/trunk/content/
--
--- cms:source-revision (original)
+++ cms:source-revision Thu May 21 02:29:12 2015
@@ -1 +1 @@
-1680640
+1680722

Modified: websites/staging/accumulo/trunk/content/people.html
==
--- websites/staging/accumulo/trunk/content/people.html (original)
+++ websites/staging/accumulo/trunk/content/people.html Thu May 21 02:29:12 2015
@@ -311,6 +311,7 @@ Latest 1.5 release: 1.5.2Tim Halloran
   Tim Reardon
   Travis Pinney
+  Trent Nadeauhttp://www.anthemengineering.com/";>Anthem Engineering LLCET 
(http://www.timeanddate.com/library/abbreviations/timezones/na/est.html>-5
 / http://www.timeanddate.com/library/abbreviations/timezones/na/edt.html>-4)
   Tristen Georgiouhttp://phemi.com";>PHEMIPT (http://www.timeanddate.com/library/abbreviations/timezones/na/pst.html>-8
 / http://www.timeanddate.com/library/abbreviations/timezones/na/pdt.html>-7)
   Vicky Kak
   Vincent Russell




svn commit: r1680722 - /accumulo/site/trunk/content/people.mdtext

2015-05-20 Thread ctubbsii
Author: ctubbsii
Date: Thu May 21 02:29:02 2015
New Revision: 1680722

URL: http://svn.apache.org/r1680722
Log:
Add Trent Nadeau to contributors.

Modified:
accumulo/site/trunk/content/people.mdtext

Modified: accumulo/site/trunk/content/people.mdtext
URL: 
http://svn.apache.org/viewvc/accumulo/site/trunk/content/people.mdtext?rev=1680722&r1=1680721&r2=1680722&view=diff
==
--- accumulo/site/trunk/content/people.mdtext (original)
+++ accumulo/site/trunk/content/people.mdtext Thu May 21 02:29:02 2015
@@ -123,6 +123,7 @@ Contributors
   Tim Halloran
   Tim Reardon
   Travis Pinney
+  Trent Nadeauhttp://www.anthemengineering.com/";>Anthem Engineering LLCET 
(http://www.timeanddate.com/library/abbreviations/timezones/na/est.html>-5
 / http://www.timeanddate.com/library/abbreviations/timezones/na/edt.html>-4)
   Tristen Georgiouhttp://phemi.com";>PHEMIPT (http://www.timeanddate.com/library/abbreviations/timezones/na/pst.html>-8
 / http://www.timeanddate.com/library/abbreviations/timezones/na/pdt.html>-7)
   Vicky Kak
   Vincent Russell




[2/6] accumulo git commit: ACCUMULO-3839 Perform sanity check on the proper replacement pair element.

2015-05-20 Thread elserj
ACCUMULO-3839 Perform sanity check on the proper replacement pair element.

The sanity check is meant to warn if a volume being
replaced is still listed in instance.volumes. The opposite
is nonsensical.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/63be7d36
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/63be7d36
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/63be7d36

Branch: refs/heads/1.7
Commit: 63be7d366ce04ef89a1d6ff3bf4af8406b3a60d5
Parents: c4eff0c
Author: Josh Elser 
Authored: Wed May 20 22:03:40 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 22:05:45 2015 -0400

--
 .../src/main/java/org/apache/accumulo/server/init/Initialize.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/63be7d36/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
--
diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java 
b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
index fd46ab7..fcc22c5 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
@@ -533,7 +533,7 @@ public class Initialize {
 Path versionPath = new Path(aBasePath, ServerConstants.VERSION_DIR);
 
 for (Pair replacementVolume : 
ServerConstants.getVolumeReplacements()) {
-  if (aBasePath.equals(replacementVolume.getSecond()))
+  if (aBasePath.equals(replacementVolume.getFirst()))
 log.error(aBasePath + " is set to be replaced in " + 
Property.INSTANCE_VOLUMES_REPLACEMENTS + " and should not appear in " +
 Property.INSTANCE_VOLUMES + ". It is highly recommended that this 
property be removed as data could still be written to this volume.");
 }



[4/6] accumulo git commit: Merge branch '1.6' into 1.7

2015-05-20 Thread elserj
Merge branch '1.6' into 1.7

Conflicts:

server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/25d9b285
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/25d9b285
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/25d9b285

Branch: refs/heads/master
Commit: 25d9b2854bd53562b67939b7b69c80924eb6e32f
Parents: 1faee53 63be7d3
Author: Josh Elser 
Authored: Wed May 20 22:26:24 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 22:26:24 2015 -0400

--
 .../src/main/java/org/apache/accumulo/server/init/Initialize.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/25d9b285/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
--
diff --cc 
server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
index c6f1dd8,fcc22c5..9b9dac9
--- a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
@@@ -701,13 -532,14 +701,13 @@@ public class Initialize implements Keyw
  Path iidPath = new Path(aBasePath, ServerConstants.INSTANCE_ID_DIR);
  Path versionPath = new Path(aBasePath, ServerConstants.VERSION_DIR);
  
 +UUID uuid = UUID.fromString(ZooUtil.getInstanceIDFromHdfs(iidPath, 
SiteConfiguration.getInstance()));
  for (Pair replacementVolume : 
ServerConstants.getVolumeReplacements()) {
-   if (aBasePath.equals(replacementVolume.getSecond()))
+   if (aBasePath.equals(replacementVolume.getFirst()))
 -log.error(aBasePath + " is set to be replaced in " + 
Property.INSTANCE_VOLUMES_REPLACEMENTS + " and should not appear in " +
 -Property.INSTANCE_VOLUMES + ". It is highly recommended that this 
property be removed as data could still be written to this volume.");
 +log.error(aBasePath + " is set to be replaced in " + 
Property.INSTANCE_VOLUMES_REPLACEMENTS + " and should not appear in " + 
Property.INSTANCE_VOLUMES
 ++ ". It is highly recommended that this property be removed as 
data could still be written to this volume.");
  }
  
 -UUID uuid = UUID.fromString(ZooUtil.getInstanceIDFromHdfs(iidPath, 
ServerConfiguration.getSiteConfiguration()));
 -
  if (ServerConstants.DATA_VERSION != 
Accumulo.getAccumuloPersistentVersion(versionPath.getFileSystem(CachedConfiguration.getInstance()),
 versionPath)) {
throw new IOException("Accumulo " + Constants.VERSION + " cannot 
initialize data version " + Accumulo.getAccumuloPersistentVersion(fs));
  }



[6/6] accumulo git commit: Merge branch '1.7'

2015-05-20 Thread elserj
Merge branch '1.7'


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/9cd82738
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/9cd82738
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/9cd82738

Branch: refs/heads/master
Commit: 9cd82738e5a1b07c709c4407d977955eed58b91f
Parents: acf6082 25d9b28
Author: Josh Elser 
Authored: Wed May 20 22:26:29 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 22:26:29 2015 -0400

--
 .../src/main/java/org/apache/accumulo/server/init/Initialize.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/9cd82738/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
--



[1/6] accumulo git commit: ACCUMULO-3839 Perform sanity check on the proper replacement pair element.

2015-05-20 Thread elserj
Repository: accumulo
Updated Branches:
  refs/heads/1.6 c4eff0c2e -> 63be7d366
  refs/heads/1.7 1faee537d -> 25d9b2854
  refs/heads/master acf60826e -> 9cd82738e


ACCUMULO-3839 Perform sanity check on the proper replacement pair element.

The sanity check is meant to warn if a volume being
replaced is still listed in instance.volumes. The opposite
is nonsensical.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/63be7d36
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/63be7d36
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/63be7d36

Branch: refs/heads/1.6
Commit: 63be7d366ce04ef89a1d6ff3bf4af8406b3a60d5
Parents: c4eff0c
Author: Josh Elser 
Authored: Wed May 20 22:03:40 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 22:05:45 2015 -0400

--
 .../src/main/java/org/apache/accumulo/server/init/Initialize.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/63be7d36/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
--
diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java 
b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
index fd46ab7..fcc22c5 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
@@ -533,7 +533,7 @@ public class Initialize {
 Path versionPath = new Path(aBasePath, ServerConstants.VERSION_DIR);
 
 for (Pair replacementVolume : 
ServerConstants.getVolumeReplacements()) {
-  if (aBasePath.equals(replacementVolume.getSecond()))
+  if (aBasePath.equals(replacementVolume.getFirst()))
 log.error(aBasePath + " is set to be replaced in " + 
Property.INSTANCE_VOLUMES_REPLACEMENTS + " and should not appear in " +
 Property.INSTANCE_VOLUMES + ". It is highly recommended that this 
property be removed as data could still be written to this volume.");
 }



[3/6] accumulo git commit: ACCUMULO-3839 Perform sanity check on the proper replacement pair element.

2015-05-20 Thread elserj
ACCUMULO-3839 Perform sanity check on the proper replacement pair element.

The sanity check is meant to warn if a volume being
replaced is still listed in instance.volumes. The opposite
is nonsensical.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/63be7d36
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/63be7d36
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/63be7d36

Branch: refs/heads/master
Commit: 63be7d366ce04ef89a1d6ff3bf4af8406b3a60d5
Parents: c4eff0c
Author: Josh Elser 
Authored: Wed May 20 22:03:40 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 22:05:45 2015 -0400

--
 .../src/main/java/org/apache/accumulo/server/init/Initialize.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/63be7d36/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
--
diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java 
b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
index fd46ab7..fcc22c5 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
@@ -533,7 +533,7 @@ public class Initialize {
 Path versionPath = new Path(aBasePath, ServerConstants.VERSION_DIR);
 
 for (Pair replacementVolume : 
ServerConstants.getVolumeReplacements()) {
-  if (aBasePath.equals(replacementVolume.getSecond()))
+  if (aBasePath.equals(replacementVolume.getFirst()))
 log.error(aBasePath + " is set to be replaced in " + 
Property.INSTANCE_VOLUMES_REPLACEMENTS + " and should not appear in " +
 Property.INSTANCE_VOLUMES + ". It is highly recommended that this 
property be removed as data could still be written to this volume.");
 }



[5/6] accumulo git commit: Merge branch '1.6' into 1.7

2015-05-20 Thread elserj
Merge branch '1.6' into 1.7

Conflicts:

server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/25d9b285
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/25d9b285
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/25d9b285

Branch: refs/heads/1.7
Commit: 25d9b2854bd53562b67939b7b69c80924eb6e32f
Parents: 1faee53 63be7d3
Author: Josh Elser 
Authored: Wed May 20 22:26:24 2015 -0400
Committer: Josh Elser 
Committed: Wed May 20 22:26:24 2015 -0400

--
 .../src/main/java/org/apache/accumulo/server/init/Initialize.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/25d9b285/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
--
diff --cc 
server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
index c6f1dd8,fcc22c5..9b9dac9
--- a/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
@@@ -701,13 -532,14 +701,13 @@@ public class Initialize implements Keyw
  Path iidPath = new Path(aBasePath, ServerConstants.INSTANCE_ID_DIR);
  Path versionPath = new Path(aBasePath, ServerConstants.VERSION_DIR);
  
 +UUID uuid = UUID.fromString(ZooUtil.getInstanceIDFromHdfs(iidPath, 
SiteConfiguration.getInstance()));
  for (Pair replacementVolume : 
ServerConstants.getVolumeReplacements()) {
-   if (aBasePath.equals(replacementVolume.getSecond()))
+   if (aBasePath.equals(replacementVolume.getFirst()))
 -log.error(aBasePath + " is set to be replaced in " + 
Property.INSTANCE_VOLUMES_REPLACEMENTS + " and should not appear in " +
 -Property.INSTANCE_VOLUMES + ". It is highly recommended that this 
property be removed as data could still be written to this volume.");
 +log.error(aBasePath + " is set to be replaced in " + 
Property.INSTANCE_VOLUMES_REPLACEMENTS + " and should not appear in " + 
Property.INSTANCE_VOLUMES
 ++ ". It is highly recommended that this property be removed as 
data could still be written to this volume.");
  }
  
 -UUID uuid = UUID.fromString(ZooUtil.getInstanceIDFromHdfs(iidPath, 
ServerConfiguration.getSiteConfiguration()));
 -
  if (ServerConstants.DATA_VERSION != 
Accumulo.getAccumuloPersistentVersion(versionPath.getFileSystem(CachedConfiguration.getInstance()),
 versionPath)) {
throw new IOException("Accumulo " + Constants.VERSION + " cannot 
initialize data version " + Accumulo.getAccumuloPersistentVersion(fs));
  }



svn commit: r952062 - in /websites/production/accumulo/content: ./ 1.4/apidocs/ 1.5/apidocs/ 1.6/apidocs/ 1.7/apidocs/

2015-05-20 Thread elserj
Author: elserj
Date: Wed May 20 17:06:59 2015
New Revision: 952062

Log:
Publishing svnmucc operation to accumulo site by elserj

Added:
websites/production/accumulo/content/
  - copied from r952061, websites/staging/accumulo/trunk/content/
websites/production/accumulo/content/1.4/apidocs/
  - copied from r952061, websites/production/accumulo/content/1.4/apidocs/
websites/production/accumulo/content/1.5/apidocs/
  - copied from r952061, websites/production/accumulo/content/1.5/apidocs/
websites/production/accumulo/content/1.6/apidocs/
  - copied from r952061, websites/production/accumulo/content/1.6/apidocs/
websites/production/accumulo/content/1.7/apidocs/
  - copied from r952061, websites/production/accumulo/content/1.7/apidocs/



svn commit: r1680640 - /accumulo/site/trunk/content/releasing.mdtext

2015-05-20 Thread elserj
Author: elserj
Date: Wed May 20 17:06:35 2015
New Revision: 1680640

URL: http://svn.apache.org/r1680640
Log:
ACCUMULO-3825 Note the addrelease form on projects.a.o

Modified:
accumulo/site/trunk/content/releasing.mdtext

Modified: accumulo/site/trunk/content/releasing.mdtext
URL: 
http://svn.apache.org/viewvc/accumulo/site/trunk/content/releasing.mdtext?rev=1680640&r1=1680639&r2=1680640&view=diff
==
--- accumulo/site/trunk/content/releasing.mdtext (original)
+++ accumulo/site/trunk/content/releasing.mdtext Wed May 20 17:06:35 2015
@@ -107,6 +107,10 @@ are releasing a bug-fix release, be sure
 1.6.2, remove 1.6.1). The old tarballs removed from dist.apache.org will still 
be preserved in archive.apache.org
 automatically.
 
+## Update projects.apache.org
+
+Fill out the [add release][addrelease] form to update the projects website.
+
 ## Update the Website
 
 After a successful vote, this website needs to be updated with the new 
artifacts.
@@ -157,3 +161,4 @@ Some good references that explain a few
 [7]: http://www.kb.cert.org/vuls/id/225657
 [8]: http://www.apache.org/dev/cmsref#extpaths
 [9]: https://svn.apache.org/repos/asf/accumulo/site/trunk
+[addrelease]: https://reporter.apache.org/addrelease.html?accumulo
\ No newline at end of file




svn commit: r952061 - in /websites/staging/accumulo/trunk/content: ./ releasing.html

2015-05-20 Thread buildbot
Author: buildbot
Date: Wed May 20 17:06:41 2015
New Revision: 952061

Log:
Staging update by buildbot for accumulo

Modified:
websites/staging/accumulo/trunk/content/   (props changed)
websites/staging/accumulo/trunk/content/releasing.html

Propchange: websites/staging/accumulo/trunk/content/
--
--- cms:source-revision (original)
+++ cms:source-revision Wed May 20 17:06:41 2015
@@ -1 +1 @@
-1680636
+1680640

Modified: websites/staging/accumulo/trunk/content/releasing.html
==
--- websites/staging/accumulo/trunk/content/releasing.html (original)
+++ websites/staging/accumulo/trunk/content/releasing.html Wed May 20 17:06:41 
2015
@@ -286,6 +286,8 @@ tarballs, the GPG signatures and checksu
 are releasing a bug-fix release, be sure to delete the previous release in the 
same line (e.g. if you release
 1.6.2, remove 1.6.1). The old tarballs removed from dist.apache.org will still 
be preserved in archive.apache.org
 automatically.
+Update projects.apache.org
+Fill out the https://reporter.apache.org/addrelease.html?accumulo";>add release 
form to update the projects website.
 Update the Website
 After a successful vote, this website needs to be updated with the new 
artifacts.
 




[4/4] accumulo git commit: Merge branch 'mac-with-existing-zookeeper' of github:tanadeau/accumulo

2015-05-20 Thread ctubbsii
Merge branch 'mac-with-existing-zookeeper' of github:tanadeau/accumulo

This closes #29

Signed-off-by: Christopher Tubbs 


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/acf60826
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/acf60826
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/acf60826

Branch: refs/heads/master
Commit: acf60826e25f943653fd7875d3b01ef43b7adf77
Parents: a9d1ad4 6c22aab
Author: Christopher Tubbs 
Authored: Wed May 20 12:50:38 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 12:50:38 2015 -0400

--
 minicluster/pom.xml |  10 ++
 .../minicluster/MiniAccumuloRunner.java |   4 +
 .../impl/MiniAccumuloClusterImpl.java   |  59 +-
 .../impl/MiniAccumuloConfigImpl.java|  37 ++-
 ...niAccumuloClusterExistingZooKeepersTest.java | 107 +++
 pom.xml |  12 +++
 6 files changed, 199 insertions(+), 30 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/acf60826/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloClusterImpl.java
--

http://git-wip-us.apache.org/repos/asf/accumulo/blob/acf60826/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloConfigImpl.java
--

http://git-wip-us.apache.org/repos/asf/accumulo/blob/acf60826/pom.xml
--



[2/4] accumulo git commit: ACCUMULO-3000: Added config option and impl to use existing ZooKeeper

2015-05-20 Thread ctubbsii
ACCUMULO-3000: Added config option and impl to use existing ZooKeeper


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/dd55a161
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/dd55a161
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/dd55a161

Branch: refs/heads/master
Commit: dd55a161bad766158ec5e876a84debeb46de2465
Parents: d353b61
Author: Trent Nadeau 
Authored: Thu Apr 16 14:35:54 2015 +
Committer: Trent Nadeau 
Committed: Fri May 15 15:19:01 2015 -0400

--
 .../minicluster/MiniAccumuloConfig.java | 13 +
 .../minicluster/MiniAccumuloRunner.java |  4 ++
 .../impl/MiniAccumuloClusterImpl.java   | 59 +++-
 .../impl/MiniAccumuloConfigImpl.java| 37 ++--
 4 files changed, 83 insertions(+), 30 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/dd55a161/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
--
diff --git 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
index b71b771..932507d 100644
--- 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
+++ 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
@@ -108,6 +108,19 @@ public class MiniAccumuloConfig {
   }
 
   /**
+   * Configure an existing ZooKeeper instance to use. Calling this method is 
optional. If not set, a new ZooKeeper instance is created.
+   *
+   * @param existingZooKeepers
+   *  Connection string for a already-running ZooKeeper instance
+   *
+   * @since 1.7.0
+   */
+  public MiniAccumuloConfig setExistingZooKeepers(String existingZooKeepers) {
+impl.setExistingZooKeepers(existingZooKeepers);
+return this;
+  }
+
+  /**
* Sets the amount of memory to use in the master process. Calling this 
method is optional. Default memory is 128M
*
* @param serverType

http://git-wip-us.apache.org/repos/asf/accumulo/blob/dd55a161/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
--
diff --git 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
index 13a75b5..1f1be44 100644
--- 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
+++ 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
@@ -75,6 +75,7 @@ public class MiniAccumuloRunner {
   private static final String NUM_T_SERVERS_PROP = "numTServers";
   private static final String DIRECTORY_PROP = "directory";
   private static final String INSTANCE_NAME_PROP = "instanceName";
+  private static final String EXISTING_ZOO_KEEPERS_PROP = "existingZooKeepers";
 
   private static void printProperties() {
 System.out.println("#mini Accumulo cluster runner properties.");
@@ -93,6 +94,7 @@ public class MiniAccumuloRunner {
 System.out.println("#" + TSERVER_MEMORY_PROP + "=128M");
 System.out.println("#" + ZOO_KEEPER_MEMORY_PROP + "=128M");
 System.out.println("#" + JDWP_ENABLED_PROP + "=false");
+System.out.println("#" + EXISTING_ZOO_KEEPERS_PROP + "=localhost:2181");
 
 System.out.println();
 System.out.println("# Configuration normally placed in accumulo-site.xml 
can be added using a site. prefix.");
@@ -167,6 +169,8 @@ public class MiniAccumuloRunner {
   
config.setZooKeeperPort(Integer.parseInt(opts.prop.getProperty(ZOO_KEEPER_PORT_PROP)));
 if (opts.prop.containsKey(ZOO_KEEPER_STARTUP_TIME_PROP))
   
config.setZooKeeperStartupTime(Long.parseLong(opts.prop.getProperty(ZOO_KEEPER_STARTUP_TIME_PROP)));
+if (opts.prop.containsKey(EXISTING_ZOO_KEEPERS_PROP))
+  
config.setExistingZooKeepers(opts.prop.getProperty(EXISTING_ZOO_KEEPERS_PROP));
 if (opts.prop.containsKey(JDWP_ENABLED_PROP))
   
config.setJDWPEnabled(Boolean.parseBoolean(opts.prop.getProperty(JDWP_ENABLED_PROP)));
 if (opts.prop.containsKey(ZOO_KEEPER_MEMORY_PROP))

http://git-wip-us.apache.org/repos/asf/accumulo/blob/dd55a161/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloClusterImpl.java
--
diff --git 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloClusterImpl.java
 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloClusterImpl.java
index a21ba64..dd20850 100644
--- 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/i

[1/4] accumulo git commit: ACCUMULO-3000: Added tests for "existing ZooKeeper" functionality

2015-05-20 Thread ctubbsii
Repository: accumulo
Updated Branches:
  refs/heads/master a9d1ad4a7 -> acf60826e


ACCUMULO-3000: Added tests for "existing ZooKeeper" functionality


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/0c069d1f
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/0c069d1f
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/0c069d1f

Branch: refs/heads/master
Commit: 0c069d1f5003fd309432765c863fb47c066a1d33
Parents: dd55a16
Author: Trent Nadeau 
Authored: Tue Apr 21 10:47:11 2015 -0400
Committer: Trent Nadeau 
Committed: Fri May 15 15:19:01 2015 -0400

--
 .gitignore  |   1 +
 minicluster/pom.xml |  10 ++
 ...niAccumuloClusterExistingZooKeepersTest.java | 107 +++
 pom.xml |  12 +++
 4 files changed, 130 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/0c069d1f/.gitignore
--
diff --git a/.gitignore b/.gitignore
index 56204d2..f7db771 100644
--- a/.gitignore
+++ b/.gitignore
@@ -23,3 +23,4 @@
 /.pydevproject
 /.idea
 /*.iml
+.*.swp

http://git-wip-us.apache.org/repos/asf/accumulo/blob/0c069d1f/minicluster/pom.xml
--
diff --git a/minicluster/pom.xml b/minicluster/pom.xml
index ff390f9..afd459c 100644
--- a/minicluster/pom.xml
+++ b/minicluster/pom.xml
@@ -105,6 +105,16 @@
   test
 
 
+  org.apache.curator
+  curator-framework
+  test
+
+
+  org.apache.curator
+  curator-test
+  test
+
+
   org.easymock
   easymock
   test

http://git-wip-us.apache.org/repos/asf/accumulo/blob/0c069d1f/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java
--
diff --git 
a/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java
 
b/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java
new file mode 100644
index 000..d9fb63e
--- /dev/null
+++ 
b/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java
@@ -0,0 +1,107 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.accumulo.minicluster;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.Map;
+
+import org.apache.accumulo.core.client.Connector;
+import org.apache.accumulo.core.client.Instance;
+import org.apache.commons.io.FileUtils;
+import org.apache.curator.framework.CuratorFramework;
+import org.apache.curator.framework.CuratorFrameworkFactory;
+import org.apache.curator.retry.RetryOneTime;
+import org.apache.curator.test.TestingServer;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.rules.TestName;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+public class MiniAccumuloClusterExistingZooKeepersTest {
+  private static final File BASE_DIR = new File(
+  System.getProperty("user.dir") + "/target/mini-tests/" + 
MiniAccumuloClusterExistingZooKeepersTest.class.getName());
+
+  private static final String SECRET = "superSecret";
+
+  private static final Logger log = 
LoggerFactory.getLogger(MiniAccumuloClusterExistingZooKeepersTest.class);
+  private TestingServer zooKeeper;
+  private MiniAccumuloCluster accumulo;
+
+  @Rule
+  public TestName testName = new TestName();
+
+  @Before
+  public void setupTestCluster() throws Exception {
+assertTrue(BASE_DIR.mkdirs() || BASE_DIR.isDirectory());
+File testDir = new File(BASE_DIR, testName.getMethodName());
+FileUtils.deleteQuietly(testDir);
+assertTrue(testDir.mkdir());
+
+ 

[3/4] accumulo git commit: ACCUMULO-3000: Use MAC config impl when using existing zookeepers

2015-05-20 Thread ctubbsii
ACCUMULO-3000: Use MAC config impl when using existing zookeepers


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/6c22aabf
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/6c22aabf
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/6c22aabf

Branch: refs/heads/master
Commit: 6c22aabfc604c6a93f215496577f9ed6ae7372bc
Parents: 0c069d1
Author: Trent Nadeau 
Authored: Tue May 12 17:36:38 2015 -0400
Committer: Trent Nadeau 
Committed: Fri May 15 15:19:02 2015 -0400

--
 .../accumulo/minicluster/MiniAccumuloConfig.java   | 13 -
 .../accumulo/minicluster/MiniAccumuloRunner.java   |  2 +-
 .../minicluster/impl/MiniAccumuloConfigImpl.java   |  2 +-
 .../MiniAccumuloClusterExistingZooKeepersTest.java |  6 +++---
 4 files changed, 5 insertions(+), 18 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/6c22aabf/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
--
diff --git 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
index 932507d..b71b771 100644
--- 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
+++ 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloConfig.java
@@ -108,19 +108,6 @@ public class MiniAccumuloConfig {
   }
 
   /**
-   * Configure an existing ZooKeeper instance to use. Calling this method is 
optional. If not set, a new ZooKeeper instance is created.
-   *
-   * @param existingZooKeepers
-   *  Connection string for a already-running ZooKeeper instance
-   *
-   * @since 1.7.0
-   */
-  public MiniAccumuloConfig setExistingZooKeepers(String existingZooKeepers) {
-impl.setExistingZooKeepers(existingZooKeepers);
-return this;
-  }
-
-  /**
* Sets the amount of memory to use in the master process. Calling this 
method is optional. Default memory is 128M
*
* @param serverType

http://git-wip-us.apache.org/repos/asf/accumulo/blob/6c22aabf/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
--
diff --git 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
index 1f1be44..78af6a0 100644
--- 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
+++ 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/MiniAccumuloRunner.java
@@ -170,7 +170,7 @@ public class MiniAccumuloRunner {
 if (opts.prop.containsKey(ZOO_KEEPER_STARTUP_TIME_PROP))
   
config.setZooKeeperStartupTime(Long.parseLong(opts.prop.getProperty(ZOO_KEEPER_STARTUP_TIME_PROP)));
 if (opts.prop.containsKey(EXISTING_ZOO_KEEPERS_PROP))
-  
config.setExistingZooKeepers(opts.prop.getProperty(EXISTING_ZOO_KEEPERS_PROP));
+  
config.getImpl().setExistingZooKeepers(opts.prop.getProperty(EXISTING_ZOO_KEEPERS_PROP));
 if (opts.prop.containsKey(JDWP_ENABLED_PROP))
   
config.setJDWPEnabled(Boolean.parseBoolean(opts.prop.getProperty(JDWP_ENABLED_PROP)));
 if (opts.prop.containsKey(ZOO_KEEPER_MEMORY_PROP))

http://git-wip-us.apache.org/repos/asf/accumulo/blob/6c22aabf/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloConfigImpl.java
--
diff --git 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloConfigImpl.java
 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloConfigImpl.java
index 25f3c99..f2c4b80 100644
--- 
a/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloConfigImpl.java
+++ 
b/minicluster/src/main/java/org/apache/accumulo/minicluster/impl/MiniAccumuloConfigImpl.java
@@ -324,7 +324,7 @@ public class MiniAccumuloConfigImpl {
* Configure an existing ZooKeeper instance to use. Calling this method is 
optional. If not set, a new ZooKeeper instance is created.
*
* @param existingZooKeepers
-   *  Connection string for a already-running ZooKeeper instance
+   *  Connection string for a already-running ZooKeeper instance. A 
null value will turn off this feature.
*
* @since 1.8.0
*/

http://git-wip-us.apache.org/repos/asf/accumulo/blob/6c22aabf/minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterExistingZooKeepersTest.java
--
diff --git 
a/minicluster/src/test/java/org/apache/accumulo/minicluster/Mini

svn commit: r952060 - in /websites/production/accumulo/content: ./ 1.4/apidocs/ 1.5/apidocs/ 1.6/apidocs/ 1.7/apidocs/

2015-05-20 Thread elserj
Author: elserj
Date: Wed May 20 16:47:44 2015
New Revision: 952060

Log:
Publishing svnmucc operation to accumulo site by elserj

Added:
websites/production/accumulo/content/
  - copied from r952059, websites/staging/accumulo/trunk/content/
websites/production/accumulo/content/1.4/apidocs/
  - copied from r952059, websites/production/accumulo/content/1.4/apidocs/
websites/production/accumulo/content/1.5/apidocs/
  - copied from r952059, websites/production/accumulo/content/1.5/apidocs/
websites/production/accumulo/content/1.6/apidocs/
  - copied from r952059, websites/production/accumulo/content/1.6/apidocs/
websites/production/accumulo/content/1.7/apidocs/
  - copied from r952059, websites/production/accumulo/content/1.7/apidocs/



svn commit: r1680636 - /accumulo/site/trunk/content/releasing.mdtext

2015-05-20 Thread elserj
Author: elserj
Date: Wed May 20 16:45:53 2015
New Revision: 1680636

URL: http://svn.apache.org/r1680636
Log:
ACCUMULO-3825 Update how to get the javadocs for a release

Modified:
accumulo/site/trunk/content/releasing.mdtext

Modified: accumulo/site/trunk/content/releasing.mdtext
URL: 
http://svn.apache.org/viewvc/accumulo/site/trunk/content/releasing.mdtext?rev=1680636&r1=1680635&r2=1680636&view=diff
==
--- accumulo/site/trunk/content/releasing.mdtext (original)
+++ accumulo/site/trunk/content/releasing.mdtext Wed May 20 16:45:53 2015
@@ -125,7 +125,7 @@ switch to Java 7). Despite the javadocs
 where they need to be committed using [extpaths][8]. You can follow the steps 
below to update
 the javadocs.
 
-1. Extract javadocs from binary tarball from the release
+1. Extract javadocs from binary tarball (1.5) or build the javadocs `mvn clean 
compile javadoc:aggregate` (>=1.6)
 2. Run the vulnerability-check tool
 3. Checkout the [Accumulo CMS repository][9]
 4. Copy the entire apidocs folder in the appropriate version folder in the CMS 
repository (e.g. content/1.5/apidocs for a 1.5 release)




svn commit: r952059 - in /websites/staging/accumulo/trunk/content: ./ releasing.html

2015-05-20 Thread buildbot
Author: buildbot
Date: Wed May 20 16:45:59 2015
New Revision: 952059

Log:
Staging update by buildbot for accumulo

Modified:
websites/staging/accumulo/trunk/content/   (props changed)
websites/staging/accumulo/trunk/content/releasing.html

Propchange: websites/staging/accumulo/trunk/content/
--
--- cms:source-revision (original)
+++ cms:source-revision Wed May 20 16:45:59 2015
@@ -1 +1 @@
-1680599
+1680636

Modified: websites/staging/accumulo/trunk/content/releasing.html
==
--- websites/staging/accumulo/trunk/content/releasing.html (original)
+++ websites/staging/accumulo/trunk/content/releasing.html Wed May 20 16:45:59 
2015
@@ -302,7 +302,7 @@ switch to Java 7). Despite the javadocs
 where they need to be committed using http://www.apache.org/dev/cmsref#extpaths";>extpaths. You can follow 
the steps below to update
 the javadocs.
 
-Extract javadocs from binary tarball from the release
+Extract javadocs from binary tarball (1.5) or build the javadocs mvn 
clean compile javadoc:aggregate (>=1.6)
 Run the vulnerability-check tool
 Checkout the https://svn.apache.org/repos/asf/accumulo/site/trunk";>Accumulo CMS 
repository
 Copy the entire apidocs folder in the appropriate version folder in the 
CMS repository (e.g. content/1.5/apidocs for a 1.5 release)




[07/21] accumulo git commit: ACCUMULO-3800 Add DEPENDENCIES file

2015-05-20 Thread ctubbsii
ACCUMULO-3800 Add DEPENDENCIES file

* Prevent a new DEPENDENCIES file from being generated by the
  maven-remote-resources-plugin's apache-jar-resource-bundle.
* Create a minimal DEPENDENCIES file which refers to the POMs.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/eef4dfe1
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/eef4dfe1
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/eef4dfe1

Branch: refs/heads/master
Commit: eef4dfe1f074ee5197ef2d10f56915c43aba81a1
Parents: dc1d0de
Author: Christopher Tubbs 
Authored: Tue May 12 12:10:28 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 12:44:08 2015 -0400

--
 DEPENDENCIES | 21 +
 1 file changed, 21 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/eef4dfe1/DEPENDENCIES
--
diff --git a/DEPENDENCIES b/DEPENDENCIES
new file mode 100644
index 000..5a98c0f
--- /dev/null
+++ b/DEPENDENCIES
@@ -0,0 +1,21 @@
+
+
+Apache Accumulo depends on artifacts which can be found in Maven Central.
+
+Each module has its own dependencies. Please refer to the individual
+modules' pom.xml files for a comprehensive listing.



[11/21] accumulo git commit: ACCUMULO-3743 One more CHANGES change.

2015-05-20 Thread ctubbsii
ACCUMULO-3743 One more CHANGES change.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/9fda3bbb
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/9fda3bbb
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/9fda3bbb

Branch: refs/heads/1.7
Commit: 9fda3bbb2c5a06764412c2ed665e47e75a9762da
Parents: 0d9c05e
Author: Josh Elser 
Authored: Tue May 12 13:42:23 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 13:42:23 2015 -0400

--
 CHANGES | 1 +
 1 file changed, 1 insertion(+)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/9fda3bbb/CHANGES
--
diff --git a/CHANGES b/CHANGES
index 7750d66..d966db4 100644
--- a/CHANGES
+++ b/CHANGES
@@ -474,6 +474,7 @@ Release Notes - Accumulo - Version 1.7.0
 * [ACCUMULO-3791] - Agitator start/stop scripts collide variable names
 * [ACCUMULO-3792] - 
org.apache.accumulo.test.randomwalk.Node#getMapReduceJars needs to include 
htrace
 * [ACCUMULO-3800] - Extra DEPENDENCIES file after new apache.pom
+* [ACCUMULO-3803] - Resurrect mapreduce.lib.util for 1.7.0
 
 ** Improvement
 * [ACCUMULO-898] - look into replacing cloudtrace



[05/21] accumulo git commit: Revert "ACCUMULO-3800 Add exclusion for DEPENDENCIES"

2015-05-20 Thread ctubbsii
Revert "ACCUMULO-3800 Add exclusion for DEPENDENCIES"

This reverts commit 8e99ed222db49e0f9a4f03ae90d1cbcf44bb1a87.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/dc1d0def
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/dc1d0def
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/dc1d0def

Branch: refs/heads/1.7
Commit: dc1d0defb79b5fde68631f1063be021fb3420f74
Parents: c16fd9d
Author: Christopher Tubbs 
Authored: Tue May 12 11:42:12 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 12:44:01 2015 -0400

--
 pom.xml | 9 -
 1 file changed, 9 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/dc1d0def/pom.xml
--
diff --git a/pom.xml b/pom.xml
index f4c734b..5bf65e8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -955,15 +955,6 @@
 
   
 
-
-  org.apache.rat
-  apache-rat-plugin
-  
-
-  DEPENDENCIES
-
-  
-
   
 
 



[08/21] accumulo git commit: ACCUMULO-3800 Add DEPENDENCIES file

2015-05-20 Thread ctubbsii
ACCUMULO-3800 Add DEPENDENCIES file

* Prevent a new DEPENDENCIES file from being generated by the
  maven-remote-resources-plugin's apache-jar-resource-bundle.
* Create a minimal DEPENDENCIES file which refers to the POMs.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/eef4dfe1
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/eef4dfe1
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/eef4dfe1

Branch: refs/heads/1.7
Commit: eef4dfe1f074ee5197ef2d10f56915c43aba81a1
Parents: dc1d0de
Author: Christopher Tubbs 
Authored: Tue May 12 12:10:28 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 12:44:08 2015 -0400

--
 DEPENDENCIES | 21 +
 1 file changed, 21 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/eef4dfe1/DEPENDENCIES
--
diff --git a/DEPENDENCIES b/DEPENDENCIES
new file mode 100644
index 000..5a98c0f
--- /dev/null
+++ b/DEPENDENCIES
@@ -0,0 +1,21 @@
+
+
+Apache Accumulo depends on artifacts which can be found in Maven Central.
+
+Each module has its own dependencies. Please refer to the individual
+modules' pom.xml files for a comprehensive listing.



[16/21] accumulo git commit: ACCUMULO-3820 Remove unused variable

2015-05-20 Thread ctubbsii
ACCUMULO-3820 Remove unused variable

* Remove warning for unused variable.
* Update comment explaining why the parameter wasn't used.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/6524b072
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/6524b072
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/6524b072

Branch: refs/heads/1.7
Commit: 6524b0720350170696c1692430ad47dd5f4bffd0
Parents: ed5b53b
Author: Christopher Tubbs 
Authored: Wed May 20 11:08:07 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 11:08:07 2015 -0400

--
 .../apache/accumulo/test/randomwalk/multitable/CopyTool.java   | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/6524b072/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
--
diff --git 
a/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
 
b/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
index b4e509c..251a064 100644
--- 
a/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
+++ 
b/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
@@ -19,9 +19,9 @@ package org.apache.accumulo.test.randomwalk.multitable;
 import java.io.IOException;
 
 import org.apache.accumulo.core.client.ClientConfiguration;
+import org.apache.accumulo.core.client.ClientConfiguration.ClientProperty;
 import org.apache.accumulo.core.client.Connector;
 import org.apache.accumulo.core.client.ZooKeeperInstance;
-import org.apache.accumulo.core.client.ClientConfiguration.ClientProperty;
 import org.apache.accumulo.core.client.admin.DelegationTokenConfig;
 import org.apache.accumulo.core.client.mapreduce.AccumuloInputFormat;
 import org.apache.accumulo.core.client.mapreduce.AccumuloOutputFormat;
@@ -74,9 +74,7 @@ public class CopyTool extends Configured implements Tool {
 final AuthenticationToken token;
 if 
(clientConf.getBoolean(ClientProperty.INSTANCE_RPC_SASL_ENABLED.getKey(), 
false)) {
   // Use the Kerberos creds to request a DelegationToken for MapReduce to 
use
-  final String keytab = args[1];
-
-  // Better be logged in. Could use the keytab, but we're already logged 
in soo..
+  // We could use the specified keytab (args[1]), but we're already logged 
in and don't need to, so we can just use the current user
   KerberosToken kt = new KerberosToken();
   try {
 UserGroupInformation user = UserGroupInformation.getCurrentUser();



[04/21] accumulo git commit: ACCUMULO-3801 moved GarbageCollectionLogger to proper module

2015-05-20 Thread ctubbsii
ACCUMULO-3801 moved GarbageCollectionLogger to proper module


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/c16fd9d1
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/c16fd9d1
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/c16fd9d1

Branch: refs/heads/master
Commit: c16fd9d1552085bf135c399e1ca32fad3737edb3
Parents: 4db7766
Author: Keith Turner 
Authored: Tue May 12 11:08:53 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 12:00:09 2015 -0400

--
 .../server/GarbageCollectionLogger.java | 116 +++
 .../server/GarbageCollectionLogger.java | 116 ---
 2 files changed, 116 insertions(+), 116 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/c16fd9d1/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
--
diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
 
b/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
new file mode 100644
index 000..389a544
--- /dev/null
+++ 
b/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
@@ -0,0 +1,116 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.accumulo.server;
+
+import java.lang.management.GarbageCollectorMXBean;
+import java.lang.management.ManagementFactory;
+import java.util.HashMap;
+import java.util.List;
+
+import org.apache.accumulo.core.conf.AccumuloConfiguration;
+import org.apache.accumulo.core.conf.Property;
+import org.apache.accumulo.server.util.Halt;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+public class GarbageCollectionLogger {
+  private static final Logger log = 
LoggerFactory.getLogger(GarbageCollectionLogger.class);
+
+  private final HashMap prevGcTime = new HashMap();
+  private long lastMemorySize = 0;
+  private long gcTimeIncreasedCount = 0;
+  private static long lastMemoryCheckTime = 0;
+
+  public GarbageCollectionLogger() {}
+
+  public synchronized void logGCInfo(AccumuloConfiguration conf) {
+final long now = System.currentTimeMillis();
+
+List gcmBeans = 
ManagementFactory.getGarbageCollectorMXBeans();
+Runtime rt = Runtime.getRuntime();
+
+StringBuilder sb = new StringBuilder("gc");
+
+boolean sawChange = false;
+
+long maxIncreaseInCollectionTime = 0;
+
+for (GarbageCollectorMXBean gcBean : gcmBeans) {
+  Long prevTime = prevGcTime.get(gcBean.getName());
+  long pt = 0;
+  if (prevTime != null) {
+pt = prevTime;
+  }
+
+  long time = gcBean.getCollectionTime();
+
+  if (time - pt != 0) {
+sawChange = true;
+  }
+
+  long increaseInCollectionTime = time - pt;
+  sb.append(String.format(" %s=%,.2f(+%,.2f) secs", gcBean.getName(), time 
/ 1000.0, increaseInCollectionTime / 1000.0));
+  maxIncreaseInCollectionTime = Math.max(increaseInCollectionTime, 
maxIncreaseInCollectionTime);
+  prevGcTime.put(gcBean.getName(), time);
+}
+
+long mem = rt.freeMemory();
+if (maxIncreaseInCollectionTime == 0) {
+  gcTimeIncreasedCount = 0;
+} else {
+  gcTimeIncreasedCount++;
+  if (gcTimeIncreasedCount > 3 && mem < rt.maxMemory() * 0.05) {
+log.warn("Running low on memory");
+gcTimeIncreasedCount = 0;
+  }
+}
+
+if (mem > lastMemorySize) {
+  sawChange = true;
+}
+
+String sign = "+";
+if (mem - lastMemorySize <= 0) {
+  sign = "";
+}
+
+sb.append(String.format(" freemem=%,d(%s%,d) totalmem=%,d", mem, sign, 
(mem - lastMemorySize), rt.totalMemory()));
+
+if (sawChange) {
+  log.debug(sb.toString());
+}
+
+final long keepAliveTimeout = 
conf.getTimeInMillis(Property.INSTANCE_ZK_TIMEOUT);
+if (lastMemoryCheckTime > 0 && lastMemoryCheckTime < now) {
+  final long diff = now - lastMemoryCheckTime;
+  if (diff > keepAliveTimeout + 1000) {
+log.warn(Str

[17/21] accumulo git commit: ACCUMULO-3819 Update findbugs and checkstyle tools

2015-05-20 Thread ctubbsii
ACCUMULO-3819 Update findbugs and checkstyle tools

* Bump findbugs and checkstyle build tools to check for more problems.
* Fix newly detected problems (mainly lack of checking for null when listing
  directory contents) to ensure the build passes.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/b577410c
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/b577410c
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/b577410c

Branch: refs/heads/1.7
Commit: b577410c677b5adb2f1c0eb1c1f8f6a9061cd0ad
Parents: 6524b07
Author: Christopher Tubbs 
Authored: Wed May 20 11:50:32 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 11:50:32 2015 -0400

--
 .../core/client/impl/ConditionalWriterImpl.java |  2 +-
 .../accumulo/core/file/rfile/bcfile/Utils.java  |  2 +-
 .../iterators/user/IntersectingIterator.java| 19 ++---
 .../mapred/AccumuloFileOutputFormatTest.java|  2 +
 .../mapreduce/AccumuloFileOutputFormatTest.java |  2 +
 .../accumulo/examples/simple/shard/Index.java   |  7 +-
 .../impl/MiniAccumuloClusterImpl.java   | 17 ++--
 .../impl/MiniAccumuloConfigImpl.java|  8 +-
 pom.xml |  6 +-
 .../accumulo/server/util/SendLogToChainsaw.java |  2 +-
 .../org/apache/accumulo/monitor/util/Table.java |  2 +-
 .../monitor/util/celltypes/NumberType.java  |  2 +-
 .../accumulo/tserver/log/LocalWALRecovery.java  | 75 -
 .../accumulo/tserver/tablet/RootFilesTest.java  |  7 +-
 .../shell/commands/FormatterCommandTest.java|  2 +-
 .../start/classloader/AccumuloClassLoader.java  |  7 +-
 .../classloader/vfs/UniqueFileReplicator.java   |  3 +-
 .../accumulo/test/continuous/TimeBinner.java|  3 +
 .../test/continuous/UndefinedAnalyzer.java  | 84 ++--
 .../test/functional/CacheTestWriter.java|  3 +
 .../apache/accumulo/test/randomwalk/Node.java   | 16 ++--
 .../apache/accumulo/test/AuditMessageIT.java|  5 +-
 22 files changed, 159 insertions(+), 117 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/b577410c/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
 
b/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
index b8375dc..24040e6 100644
--- 
a/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
+++ 
b/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
@@ -182,7 +182,7 @@ class ConditionalWriterImpl implements ConditionalWriter {
 @Override
 public int compareTo(Delayed o) {
   QCMutation oqcm = (QCMutation) o;
-  return Long.valueOf(resetTime).compareTo(Long.valueOf(oqcm.resetTime));
+  return Long.compare(resetTime, oqcm.resetTime);
 }
 
 @Override

http://git-wip-us.apache.org/repos/asf/accumulo/blob/b577410c/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java 
b/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
index 6cb04a1..5e84f10 100644
--- a/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
+++ b/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
@@ -351,7 +351,7 @@ public final class Utils {
 
 @Override
 public int hashCode() {
-  return (major << 16 + minor);
+  return ((major << 16) + minor);
 }
   }
 

http://git-wip-us.apache.org/repos/asf/accumulo/blob/b577410c/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
 
b/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
index 63d6a34..e7338f3 100644
--- 
a/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
+++ 
b/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
@@ -232,19 +232,20 @@ public class IntersectingIterator implements 
SortedKeyValueIterator {
 // If we are past the target, this is a valid result
 if (docIDCompare < 0) {
   break;
-}
-// if this source is not yet at the currentCQ then advance in this 
source
-if (docIDCompare > 0) {
+} else if (docIDCompare > 0) {
+  // if this source is not yet at the currentCQ then advance in this 

[01/21] accumulo git commit: Revert "ACCUMULO-3793 use numctl to turn on memory interleaving"

2015-05-20 Thread ctubbsii
Repository: accumulo
Updated Branches:
  refs/heads/1.7 ed5b53b93 -> 1faee537d
  refs/heads/master e22103ca9 -> a9d1ad4a7


Revert "ACCUMULO-3793 use numctl to turn on memory interleaving"

This reverts commit b82148777c023b21578d06ef451fdd70770107a5.

Conflicts:
assemble/bin/config.sh
assemble/bin/start-server.sh


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/4db7766d
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/4db7766d
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/4db7766d

Branch: refs/heads/1.7
Commit: 4db7766d77497c4ab8b8cbd997fe1b4c267308f3
Parents: 8e99ed2
Author: Josh Elser 
Authored: Tue May 12 11:59:11 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 11:59:11 2015 -0400

--
 assemble/bin/config.sh   | 9 -
 assemble/bin/start-server.sh | 8 
 2 files changed, 4 insertions(+), 13 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/4db7766d/assemble/bin/config.sh
--
diff --git a/assemble/bin/config.sh b/assemble/bin/config.sh
index ed393bb..d9bf8d4 100755
--- a/assemble/bin/config.sh
+++ b/assemble/bin/config.sh
@@ -104,15 +104,6 @@ then
 fi
 export HADOOP_PREFIX
 
-NUMA=`which numactl 2>/dev/null`
-NUMACTL_EXISTS="$?"
-NUMACTL_ARGS="--interleave=all"
-if [[ ${NUMACTL_EXISTS} -eq 0 ]] ; then
-  export NUMA_CMD="${NUMA} ${NUMACTL_ARGS}"
-else
-  export NUMA_CMD=""
-fi
-
 export HADOOP_HOME=$HADOOP_PREFIX
 export HADOOP_HOME_WARN_SUPPRESS=true
 

http://git-wip-us.apache.org/repos/asf/accumulo/blob/4db7766d/assemble/bin/start-server.sh
--
diff --git a/assemble/bin/start-server.sh b/assemble/bin/start-server.sh
index 3587ec9..2fb4c4c 100755
--- a/assemble/bin/start-server.sh
+++ b/assemble/bin/start-server.sh
@@ -77,12 +77,12 @@ if [[ -z "$PID" ]]; then
   COMMAND="${bin}/accumulo_watcher.sh ${LOGHOST}"
fi
 
-   if [[ $HOST == localhost || $HOST == "$(hostname -f)" || $HOST = "$IP" ]]; 
then
-  ${NUMA_CMD} "$COMMAND" "${SERVICE}" --address "${ADDRESS}" 
>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out" 
2>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err" & 
+   if [ "$HOST" = "localhost" -o "$HOST" = "`hostname -f`" -o "$HOST" = "$ip" 
]; then
+  ${bin}/accumulo ${SERVICE} --address ${ADDRESS} 
>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err & 
   MAX_FILES_OPEN=$(ulimit -n)
else
-  $SSH "$HOST" "bash -c 'exec nohup ${NUMA_CMD} $COMMAND ${SERVICE} 
--address ${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
-  MAX_FILES_OPEN=$($SSH "$HOST" "/usr/bin/env bash -c 'ulimit -n'") 
+  $SSH $HOST "bash -c 'exec nohup ${bin}/accumulo ${SERVICE} --address 
${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
+  MAX_FILES_OPEN=$($SSH $HOST "/usr/bin/env bash -c 'ulimit -n'") 
fi
 
if [[ -n $MAX_FILES_OPEN && -n $SLAVES ]] ; then



[19/21] accumulo git commit: Merge tag '1.7.0' into 1.7

2015-05-20 Thread ctubbsii
Merge tag '1.7.0' into 1.7

Apache Accumulo 1.7.0

Conflicts:
assemble/pom.xml
core/pom.xml
docs/pom.xml
examples/simple/pom.xml
fate/pom.xml
maven-plugin/pom.xml
minicluster/pom.xml
pom.xml
proxy/pom.xml
server/base/pom.xml
server/gc/pom.xml
server/master/pom.xml
server/monitor/pom.xml
server/native/pom.xml
server/tracer/pom.xml
server/tserver/pom.xml
shell/pom.xml
start/pom.xml
test/pom.xml
trace/pom.xml


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/1faee537
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/1faee537
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/1faee537

Branch: refs/heads/1.7
Commit: 1faee537d8d557297d20f3d084b4e16127d00600
Parents: b577410 8cba812
Author: Christopher Tubbs 
Authored: Wed May 20 12:07:59 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 12:07:59 2015 -0400

--
 CHANGES  | 1 +
 assemble/bin/config.sh   | 6 +++---
 assemble/bin/start-server.sh | 4 ++--
 pom.xml  | 2 +-
 4 files changed, 7 insertions(+), 6 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/1faee537/assemble/bin/config.sh
--
diff --cc assemble/bin/config.sh
index ed393bb,d9bf8d4..9dfd164
--- a/assemble/bin/config.sh
+++ b/assemble/bin/config.sh
@@@ -104,15 -104,6 +104,15 @@@ the
  fi
  export HADOOP_PREFIX
  
- NUMA=`which numactl 2>/dev/null`
- NUMACTL_EXISTS="$?"
- NUMACTL_ARGS="--interleave=all"
++NUMA=$(which numactl 2>/dev/null)
++NUMACTL_EXISTS=$?
++NUMACTL_ARGS='--interleave=all'
 +if [[ ${NUMACTL_EXISTS} -eq 0 ]] ; then
 +  export NUMA_CMD="${NUMA} ${NUMACTL_ARGS}"
 +else
 +  export NUMA_CMD=""
 +fi
 +
  export HADOOP_HOME=$HADOOP_PREFIX
  export HADOOP_HOME_WARN_SUPPRESS=true
  

http://git-wip-us.apache.org/repos/asf/accumulo/blob/1faee537/assemble/bin/start-server.sh
--
diff --cc assemble/bin/start-server.sh
index 3587ec9,2fb4c4c..8266457
--- a/assemble/bin/start-server.sh
+++ b/assemble/bin/start-server.sh
@@@ -77,12 -77,12 +77,12 @@@ if [[ -z "$PID" ]]; the
COMMAND="${bin}/accumulo_watcher.sh ${LOGHOST}"
 fi
  
 -   if [ "$HOST" = "localhost" -o "$HOST" = "`hostname -f`" -o "$HOST" = "$ip" 
]; then
 -  ${bin}/accumulo ${SERVICE} --address ${ADDRESS} 
>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err & 
 +   if [[ $HOST == localhost || $HOST == "$(hostname -f)" || $HOST = "$IP" ]]; 
then
-   ${NUMA_CMD} "$COMMAND" "${SERVICE}" --address "${ADDRESS}" 
>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out" 
2>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err" & 
++  ${NUMA_CMD} "$COMMAND" "${SERVICE}" --address "${ADDRESS}" 
>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out" 
2>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err" &
MAX_FILES_OPEN=$(ulimit -n)
 else
 -  $SSH $HOST "bash -c 'exec nohup ${bin}/accumulo ${SERVICE} --address 
${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
 -  MAX_FILES_OPEN=$($SSH $HOST "/usr/bin/env bash -c 'ulimit -n'") 
 +  $SSH "$HOST" "bash -c 'exec nohup ${NUMA_CMD} $COMMAND ${SERVICE} 
--address ${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
-   MAX_FILES_OPEN=$($SSH "$HOST" "/usr/bin/env bash -c 'ulimit -n'") 
++  MAX_FILES_OPEN=$($SSH "$HOST" "/usr/bin/env bash -c 'ulimit -n'")
 fi
  
 if [[ -n $MAX_FILES_OPEN && -n $SLAVES ]] ; then

http://git-wip-us.apache.org/repos/asf/accumulo/blob/1faee537/pom.xml
--
diff --cc pom.xml
index 19ff679,24f25c9..3815ad7
--- a/pom.xml
+++ b/pom.xml
@@@ -95,7 -95,7 +95,7 @@@

  scm:git:git://git.apache.org/accumulo.git
  
scm:git:https://git-wip-us.apache.org/repos/asf/accumulo.git
- ${project.version}
 -1.7.0
++HEAD
  https://git-wip-us.apache.org/repos/asf?p=accumulo.git





[18/21] accumulo git commit: ACCUMULO-3819 Update findbugs and checkstyle tools

2015-05-20 Thread ctubbsii
ACCUMULO-3819 Update findbugs and checkstyle tools

* Bump findbugs and checkstyle build tools to check for more problems.
* Fix newly detected problems (mainly lack of checking for null when listing
  directory contents) to ensure the build passes.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/b577410c
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/b577410c
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/b577410c

Branch: refs/heads/master
Commit: b577410c677b5adb2f1c0eb1c1f8f6a9061cd0ad
Parents: 6524b07
Author: Christopher Tubbs 
Authored: Wed May 20 11:50:32 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 11:50:32 2015 -0400

--
 .../core/client/impl/ConditionalWriterImpl.java |  2 +-
 .../accumulo/core/file/rfile/bcfile/Utils.java  |  2 +-
 .../iterators/user/IntersectingIterator.java| 19 ++---
 .../mapred/AccumuloFileOutputFormatTest.java|  2 +
 .../mapreduce/AccumuloFileOutputFormatTest.java |  2 +
 .../accumulo/examples/simple/shard/Index.java   |  7 +-
 .../impl/MiniAccumuloClusterImpl.java   | 17 ++--
 .../impl/MiniAccumuloConfigImpl.java|  8 +-
 pom.xml |  6 +-
 .../accumulo/server/util/SendLogToChainsaw.java |  2 +-
 .../org/apache/accumulo/monitor/util/Table.java |  2 +-
 .../monitor/util/celltypes/NumberType.java  |  2 +-
 .../accumulo/tserver/log/LocalWALRecovery.java  | 75 -
 .../accumulo/tserver/tablet/RootFilesTest.java  |  7 +-
 .../shell/commands/FormatterCommandTest.java|  2 +-
 .../start/classloader/AccumuloClassLoader.java  |  7 +-
 .../classloader/vfs/UniqueFileReplicator.java   |  3 +-
 .../accumulo/test/continuous/TimeBinner.java|  3 +
 .../test/continuous/UndefinedAnalyzer.java  | 84 ++--
 .../test/functional/CacheTestWriter.java|  3 +
 .../apache/accumulo/test/randomwalk/Node.java   | 16 ++--
 .../apache/accumulo/test/AuditMessageIT.java|  5 +-
 22 files changed, 159 insertions(+), 117 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/b577410c/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
 
b/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
index b8375dc..24040e6 100644
--- 
a/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
+++ 
b/core/src/main/java/org/apache/accumulo/core/client/impl/ConditionalWriterImpl.java
@@ -182,7 +182,7 @@ class ConditionalWriterImpl implements ConditionalWriter {
 @Override
 public int compareTo(Delayed o) {
   QCMutation oqcm = (QCMutation) o;
-  return Long.valueOf(resetTime).compareTo(Long.valueOf(oqcm.resetTime));
+  return Long.compare(resetTime, oqcm.resetTime);
 }
 
 @Override

http://git-wip-us.apache.org/repos/asf/accumulo/blob/b577410c/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java 
b/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
index 6cb04a1..5e84f10 100644
--- a/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
+++ b/core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/Utils.java
@@ -351,7 +351,7 @@ public final class Utils {
 
 @Override
 public int hashCode() {
-  return (major << 16 + minor);
+  return ((major << 16) + minor);
 }
   }
 

http://git-wip-us.apache.org/repos/asf/accumulo/blob/b577410c/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
 
b/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
index 63d6a34..e7338f3 100644
--- 
a/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
+++ 
b/core/src/main/java/org/apache/accumulo/core/iterators/user/IntersectingIterator.java
@@ -232,19 +232,20 @@ public class IntersectingIterator implements 
SortedKeyValueIterator {
 // If we are past the target, this is a valid result
 if (docIDCompare < 0) {
   break;
-}
-// if this source is not yet at the currentCQ then advance in this 
source
-if (docIDCompare > 0) {
+} else if (docIDCompare > 0) {
+  // if this source is not yet at the currentCQ then advance in thi

[03/21] accumulo git commit: ACCUMULO-3801 moved GarbageCollectionLogger to proper module

2015-05-20 Thread ctubbsii
ACCUMULO-3801 moved GarbageCollectionLogger to proper module


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/c16fd9d1
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/c16fd9d1
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/c16fd9d1

Branch: refs/heads/1.7
Commit: c16fd9d1552085bf135c399e1ca32fad3737edb3
Parents: 4db7766
Author: Keith Turner 
Authored: Tue May 12 11:08:53 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 12:00:09 2015 -0400

--
 .../server/GarbageCollectionLogger.java | 116 +++
 .../server/GarbageCollectionLogger.java | 116 ---
 2 files changed, 116 insertions(+), 116 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/c16fd9d1/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
--
diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
 
b/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
new file mode 100644
index 000..389a544
--- /dev/null
+++ 
b/server/base/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
@@ -0,0 +1,116 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.accumulo.server;
+
+import java.lang.management.GarbageCollectorMXBean;
+import java.lang.management.ManagementFactory;
+import java.util.HashMap;
+import java.util.List;
+
+import org.apache.accumulo.core.conf.AccumuloConfiguration;
+import org.apache.accumulo.core.conf.Property;
+import org.apache.accumulo.server.util.Halt;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+public class GarbageCollectionLogger {
+  private static final Logger log = 
LoggerFactory.getLogger(GarbageCollectionLogger.class);
+
+  private final HashMap prevGcTime = new HashMap();
+  private long lastMemorySize = 0;
+  private long gcTimeIncreasedCount = 0;
+  private static long lastMemoryCheckTime = 0;
+
+  public GarbageCollectionLogger() {}
+
+  public synchronized void logGCInfo(AccumuloConfiguration conf) {
+final long now = System.currentTimeMillis();
+
+List gcmBeans = 
ManagementFactory.getGarbageCollectorMXBeans();
+Runtime rt = Runtime.getRuntime();
+
+StringBuilder sb = new StringBuilder("gc");
+
+boolean sawChange = false;
+
+long maxIncreaseInCollectionTime = 0;
+
+for (GarbageCollectorMXBean gcBean : gcmBeans) {
+  Long prevTime = prevGcTime.get(gcBean.getName());
+  long pt = 0;
+  if (prevTime != null) {
+pt = prevTime;
+  }
+
+  long time = gcBean.getCollectionTime();
+
+  if (time - pt != 0) {
+sawChange = true;
+  }
+
+  long increaseInCollectionTime = time - pt;
+  sb.append(String.format(" %s=%,.2f(+%,.2f) secs", gcBean.getName(), time 
/ 1000.0, increaseInCollectionTime / 1000.0));
+  maxIncreaseInCollectionTime = Math.max(increaseInCollectionTime, 
maxIncreaseInCollectionTime);
+  prevGcTime.put(gcBean.getName(), time);
+}
+
+long mem = rt.freeMemory();
+if (maxIncreaseInCollectionTime == 0) {
+  gcTimeIncreasedCount = 0;
+} else {
+  gcTimeIncreasedCount++;
+  if (gcTimeIncreasedCount > 3 && mem < rt.maxMemory() * 0.05) {
+log.warn("Running low on memory");
+gcTimeIncreasedCount = 0;
+  }
+}
+
+if (mem > lastMemorySize) {
+  sawChange = true;
+}
+
+String sign = "+";
+if (mem - lastMemorySize <= 0) {
+  sign = "";
+}
+
+sb.append(String.format(" freemem=%,d(%s%,d) totalmem=%,d", mem, sign, 
(mem - lastMemorySize), rt.totalMemory()));
+
+if (sawChange) {
+  log.debug(sb.toString());
+}
+
+final long keepAliveTimeout = 
conf.getTimeInMillis(Property.INSTANCE_ZK_TIMEOUT);
+if (lastMemoryCheckTime > 0 && lastMemoryCheckTime < now) {
+  final long diff = now - lastMemoryCheckTime;
+  if (diff > keepAliveTimeout + 1000) {
+log.warn(String

[21/21] accumulo git commit: Merge branch '1.7'

2015-05-20 Thread ctubbsii
Merge branch '1.7'


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/a9d1ad4a
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/a9d1ad4a
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/a9d1ad4a

Branch: refs/heads/master
Commit: a9d1ad4a7d470071e59382dfad0e10d49c9ee464
Parents: e22103c 1faee53
Author: Christopher Tubbs 
Authored: Wed May 20 12:09:26 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 12:09:26 2015 -0400

--
 CHANGES |  1 +
 assemble/bin/config.sh  |  6 +-
 assemble/bin/start-server.sh|  4 +-
 .../core/client/impl/ConditionalWriterImpl.java |  2 +-
 .../accumulo/core/file/rfile/bcfile/Utils.java  |  2 +-
 .../iterators/user/IntersectingIterator.java| 19 ++---
 .../mapred/AccumuloFileOutputFormatTest.java|  2 +
 .../mapreduce/AccumuloFileOutputFormatTest.java |  2 +
 .../accumulo/examples/simple/shard/Index.java   |  7 +-
 .../impl/MiniAccumuloClusterImpl.java   | 17 ++--
 .../impl/MiniAccumuloConfigImpl.java|  8 +-
 pom.xml |  8 +-
 .../accumulo/server/util/SendLogToChainsaw.java |  2 +-
 .../org/apache/accumulo/monitor/util/Table.java |  2 +-
 .../monitor/util/celltypes/NumberType.java  |  2 +-
 .../accumulo/tserver/log/LocalWALRecovery.java  | 75 -
 .../accumulo/tserver/tablet/RootFilesTest.java  |  7 +-
 .../shell/commands/FormatterCommandTest.java|  2 +-
 .../start/classloader/AccumuloClassLoader.java  |  7 +-
 .../classloader/vfs/UniqueFileReplicator.java   |  3 +-
 .../accumulo/test/continuous/TimeBinner.java|  3 +
 .../test/continuous/UndefinedAnalyzer.java  | 84 ++--
 .../test/functional/CacheTestWriter.java|  3 +
 .../apache/accumulo/test/randomwalk/Node.java   | 16 ++--
 .../test/randomwalk/multitable/CopyTool.java|  6 +-
 .../apache/accumulo/test/AuditMessageIT.java|  5 +-
 26 files changed, 168 insertions(+), 127 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/a9d1ad4a/pom.xml
--



[02/21] accumulo git commit: Revert "ACCUMULO-3793 use numctl to turn on memory interleaving"

2015-05-20 Thread ctubbsii
Revert "ACCUMULO-3793 use numctl to turn on memory interleaving"

This reverts commit b82148777c023b21578d06ef451fdd70770107a5.

Conflicts:
assemble/bin/config.sh
assemble/bin/start-server.sh


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/4db7766d
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/4db7766d
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/4db7766d

Branch: refs/heads/master
Commit: 4db7766d77497c4ab8b8cbd997fe1b4c267308f3
Parents: 8e99ed2
Author: Josh Elser 
Authored: Tue May 12 11:59:11 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 11:59:11 2015 -0400

--
 assemble/bin/config.sh   | 9 -
 assemble/bin/start-server.sh | 8 
 2 files changed, 4 insertions(+), 13 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/4db7766d/assemble/bin/config.sh
--
diff --git a/assemble/bin/config.sh b/assemble/bin/config.sh
index ed393bb..d9bf8d4 100755
--- a/assemble/bin/config.sh
+++ b/assemble/bin/config.sh
@@ -104,15 +104,6 @@ then
 fi
 export HADOOP_PREFIX
 
-NUMA=`which numactl 2>/dev/null`
-NUMACTL_EXISTS="$?"
-NUMACTL_ARGS="--interleave=all"
-if [[ ${NUMACTL_EXISTS} -eq 0 ]] ; then
-  export NUMA_CMD="${NUMA} ${NUMACTL_ARGS}"
-else
-  export NUMA_CMD=""
-fi
-
 export HADOOP_HOME=$HADOOP_PREFIX
 export HADOOP_HOME_WARN_SUPPRESS=true
 

http://git-wip-us.apache.org/repos/asf/accumulo/blob/4db7766d/assemble/bin/start-server.sh
--
diff --git a/assemble/bin/start-server.sh b/assemble/bin/start-server.sh
index 3587ec9..2fb4c4c 100755
--- a/assemble/bin/start-server.sh
+++ b/assemble/bin/start-server.sh
@@ -77,12 +77,12 @@ if [[ -z "$PID" ]]; then
   COMMAND="${bin}/accumulo_watcher.sh ${LOGHOST}"
fi
 
-   if [[ $HOST == localhost || $HOST == "$(hostname -f)" || $HOST = "$IP" ]]; 
then
-  ${NUMA_CMD} "$COMMAND" "${SERVICE}" --address "${ADDRESS}" 
>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out" 
2>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err" & 
+   if [ "$HOST" = "localhost" -o "$HOST" = "`hostname -f`" -o "$HOST" = "$ip" 
]; then
+  ${bin}/accumulo ${SERVICE} --address ${ADDRESS} 
>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err & 
   MAX_FILES_OPEN=$(ulimit -n)
else
-  $SSH "$HOST" "bash -c 'exec nohup ${NUMA_CMD} $COMMAND ${SERVICE} 
--address ${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
-  MAX_FILES_OPEN=$($SSH "$HOST" "/usr/bin/env bash -c 'ulimit -n'") 
+  $SSH $HOST "bash -c 'exec nohup ${bin}/accumulo ${SERVICE} --address 
${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
+  MAX_FILES_OPEN=$($SSH $HOST "/usr/bin/env bash -c 'ulimit -n'") 
fi
 
if [[ -n $MAX_FILES_OPEN && -n $SLAVES ]] ; then



[14/21] accumulo git commit: [maven-release-plugin] prepare release 1.7.0

2015-05-20 Thread ctubbsii
[maven-release-plugin] prepare release 1.7.0


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/8cba8128
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/8cba8128
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/8cba8128

Branch: refs/heads/master
Commit: 8cba8128fbc3238bdd9398cf5c36b7cb6dc3b61d
Parents: 9fda3bb
Author: Josh Elser 
Authored: Tue May 12 14:34:29 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 14:34:29 2015 -0400

--
 assemble/pom.xml| 2 +-
 core/pom.xml| 2 +-
 docs/pom.xml| 2 +-
 examples/simple/pom.xml | 2 +-
 fate/pom.xml| 2 +-
 maven-plugin/pom.xml| 2 +-
 minicluster/pom.xml | 2 +-
 pom.xml | 4 ++--
 proxy/pom.xml   | 2 +-
 server/base/pom.xml | 2 +-
 server/gc/pom.xml   | 2 +-
 server/master/pom.xml   | 2 +-
 server/monitor/pom.xml  | 2 +-
 server/native/pom.xml   | 2 +-
 server/tracer/pom.xml   | 2 +-
 server/tserver/pom.xml  | 2 +-
 shell/pom.xml   | 2 +-
 start/pom.xml   | 2 +-
 test/pom.xml| 2 +-
 trace/pom.xml   | 2 +-
 20 files changed, 21 insertions(+), 21 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/assemble/pom.xml
--
diff --git a/assemble/pom.xml b/assemble/pom.xml
index e88cd81..0cf690b 100644
--- a/assemble/pom.xml
+++ b/assemble/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo
   pom

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/core/pom.xml
--
diff --git a/core/pom.xml b/core/pom.xml
index 12e3c1d..fe91d0a 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-core
   Core

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/docs/pom.xml
--
diff --git a/docs/pom.xml b/docs/pom.xml
index 47d98da..519f534 100644
--- a/docs/pom.xml
+++ b/docs/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-docs
   pom

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/examples/simple/pom.xml
--
diff --git a/examples/simple/pom.xml b/examples/simple/pom.xml
index aa2f118..7172831 100644
--- a/examples/simple/pom.xml
+++ b/examples/simple/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
 ../../pom.xml
   
   accumulo-examples-simple

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/fate/pom.xml
--
diff --git a/fate/pom.xml b/fate/pom.xml
index 5fd9a7e..e282a14 100644
--- a/fate/pom.xml
+++ b/fate/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-fate
   Fate

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/maven-plugin/pom.xml
--
diff --git a/maven-plugin/pom.xml b/maven-plugin/pom.xml
index 2c07d7d..ecb3bd0 100644
--- a/maven-plugin/pom.xml
+++ b/maven-plugin/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-maven-plugin
   maven-plugin

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/minicluster/pom.xml
--
diff --git a/minicluster/pom.xml b/minicluster/pom.xml
index 7644269..c12c967 100644
--- a/minicluster/pom.xml
+++ b/minicluster/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-minicluster
   MiniCluster

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/pom.xml
--
diff --git a/pom.xml b/pom.xml
index 5bf65e8..24f25c9 100644
--- a/pom.xml
+++ b/pom.xml
@@ -24,7 +24,7 @@
   
   org.apache.accumulo
   accumulo-project
-  1.7.0-SNAPSHOT
+  1.7.0
   pom
   Apache Accumulo
   Apache Accumulo is a sorted, distributed key/value store based 
on Google's BigTable design. It is built on top of Apache Hadoop, Zookeeper, 
and Thrift. It features a few novel improvements on the BigTable design in the 
form of cell-level access labels and a server-side programming mechanism that 
can modify key/value pairs at various points in the data management 
process.
@@ 

[15/21] accumulo git commit: ACCUMULO-3820 Remove unused variable

2015-05-20 Thread ctubbsii
ACCUMULO-3820 Remove unused variable

* Remove warning for unused variable.
* Update comment explaining why the parameter wasn't used.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/6524b072
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/6524b072
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/6524b072

Branch: refs/heads/master
Commit: 6524b0720350170696c1692430ad47dd5f4bffd0
Parents: ed5b53b
Author: Christopher Tubbs 
Authored: Wed May 20 11:08:07 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 11:08:07 2015 -0400

--
 .../apache/accumulo/test/randomwalk/multitable/CopyTool.java   | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/6524b072/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
--
diff --git 
a/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
 
b/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
index b4e509c..251a064 100644
--- 
a/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
+++ 
b/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
@@ -19,9 +19,9 @@ package org.apache.accumulo.test.randomwalk.multitable;
 import java.io.IOException;
 
 import org.apache.accumulo.core.client.ClientConfiguration;
+import org.apache.accumulo.core.client.ClientConfiguration.ClientProperty;
 import org.apache.accumulo.core.client.Connector;
 import org.apache.accumulo.core.client.ZooKeeperInstance;
-import org.apache.accumulo.core.client.ClientConfiguration.ClientProperty;
 import org.apache.accumulo.core.client.admin.DelegationTokenConfig;
 import org.apache.accumulo.core.client.mapreduce.AccumuloInputFormat;
 import org.apache.accumulo.core.client.mapreduce.AccumuloOutputFormat;
@@ -74,9 +74,7 @@ public class CopyTool extends Configured implements Tool {
 final AuthenticationToken token;
 if 
(clientConf.getBoolean(ClientProperty.INSTANCE_RPC_SASL_ENABLED.getKey(), 
false)) {
   // Use the Kerberos creds to request a DelegationToken for MapReduce to 
use
-  final String keytab = args[1];
-
-  // Better be logged in. Could use the keytab, but we're already logged 
in soo..
+  // We could use the specified keytab (args[1]), but we're already logged 
in and don't need to, so we can just use the current user
   KerberosToken kt = new KerberosToken();
   try {
 UserGroupInformation user = UserGroupInformation.getCurrentUser();



[09/21] accumulo git commit: ACCUMULO-3803 Resurected mapreduce.lib.util

2015-05-20 Thread ctubbsii
ACCUMULO-3803 Resurected mapreduce.lib.util

Ran the following command in the 1.7 branch to do this.  The commit below is 
what was at head of the 1.6 branch at the time I ran this command.

git checkout c4eff0c2eb1320e411ac3e41b6f2db89c2d3ba33 -- 
core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util

I checked in the 1.6 branch and found that this code has no references (like 
test).


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/0d9c05e7
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/0d9c05e7
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/0d9c05e7

Branch: refs/heads/1.7
Commit: 0d9c05e716774f7904c7da2664063a778ab4640b
Parents: eef4dfe
Author: Keith Turner 
Authored: Tue May 12 12:51:18 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 13:40:37 2015 -0400

--
 .../mapreduce/lib/util/ConfiguratorBase.java| 277 +++
 .../lib/util/FileOutputConfigurator.java| 170 +++
 .../mapreduce/lib/util/InputConfigurator.java   | 462 +++
 .../mapreduce/lib/util/OutputConfigurator.java  | 196 
 .../client/mapreduce/lib/util/package-info.java |  22 +
 5 files changed, 1127 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/0d9c05e7/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
 
b/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
new file mode 100644
index 000..20fbbea
--- /dev/null
+++ 
b/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
@@ -0,0 +1,277 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.accumulo.core.client.mapreduce.lib.util;
+
+import org.apache.accumulo.core.client.AccumuloSecurityException;
+import org.apache.accumulo.core.client.ClientConfiguration;
+import org.apache.accumulo.core.client.Instance;
+import org.apache.accumulo.core.client.ZooKeeperInstance;
+import org.apache.accumulo.core.client.mock.MockInstance;
+import org.apache.accumulo.core.client.security.tokens.AuthenticationToken;
+import 
org.apache.accumulo.core.client.security.tokens.AuthenticationToken.AuthenticationTokenSerializer;
+import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.util.StringUtils;
+import org.apache.log4j.Level;
+
+/**
+ * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+ * @since 1.5.0
+ */
+@Deprecated
+public class ConfiguratorBase {
+
+  /**
+   * Configuration keys for {@link Instance#getConnector(String, 
AuthenticationToken)}.
+   *
+   * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+   * @since 1.5.0
+   */
+  @Deprecated
+  public static enum ConnectorInfo {
+IS_CONFIGURED, PRINCIPAL, TOKEN, TOKEN_CLASS
+  }
+
+  /**
+   * Configuration keys for {@link Instance}, {@link ZooKeeperInstance}, and 
{@link MockInstance}.
+   *
+   * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+   * @since 1.5.0
+   */
+  @Deprecated
+  protected static enum InstanceOpts {
+TYPE, NAME, ZOO_KEEPERS;
+  }
+
+  /**
+   * Configuration keys for general configuration options.
+   *
+   * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+   * @since 1.5.0
+   */
+  @Deprecated
+  protected static enum GeneralOpts {
+LOG_LEVEL
+  }
+
+  /**
+   * Provides a configuration key for a given feature enum, prefixed by the 
implementingClass
+   *
+   * @param implementingClass
+   *  the class whose name will be used as a prefix for the property 
configuration key
+   * @param e
+   *  the enum used to provide the unique part of the configuration key
+   * @return the configuration key
+   * @deprecate

[20/21] accumulo git commit: Merge tag '1.7.0' into 1.7

2015-05-20 Thread ctubbsii
Merge tag '1.7.0' into 1.7

Apache Accumulo 1.7.0

Conflicts:
assemble/pom.xml
core/pom.xml
docs/pom.xml
examples/simple/pom.xml
fate/pom.xml
maven-plugin/pom.xml
minicluster/pom.xml
pom.xml
proxy/pom.xml
server/base/pom.xml
server/gc/pom.xml
server/master/pom.xml
server/monitor/pom.xml
server/native/pom.xml
server/tracer/pom.xml
server/tserver/pom.xml
shell/pom.xml
start/pom.xml
test/pom.xml
trace/pom.xml


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/1faee537
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/1faee537
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/1faee537

Branch: refs/heads/master
Commit: 1faee537d8d557297d20f3d084b4e16127d00600
Parents: b577410 8cba812
Author: Christopher Tubbs 
Authored: Wed May 20 12:07:59 2015 -0400
Committer: Christopher Tubbs 
Committed: Wed May 20 12:07:59 2015 -0400

--
 CHANGES  | 1 +
 assemble/bin/config.sh   | 6 +++---
 assemble/bin/start-server.sh | 4 ++--
 pom.xml  | 2 +-
 4 files changed, 7 insertions(+), 6 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/1faee537/assemble/bin/config.sh
--
diff --cc assemble/bin/config.sh
index ed393bb,d9bf8d4..9dfd164
--- a/assemble/bin/config.sh
+++ b/assemble/bin/config.sh
@@@ -104,15 -104,6 +104,15 @@@ the
  fi
  export HADOOP_PREFIX
  
- NUMA=`which numactl 2>/dev/null`
- NUMACTL_EXISTS="$?"
- NUMACTL_ARGS="--interleave=all"
++NUMA=$(which numactl 2>/dev/null)
++NUMACTL_EXISTS=$?
++NUMACTL_ARGS='--interleave=all'
 +if [[ ${NUMACTL_EXISTS} -eq 0 ]] ; then
 +  export NUMA_CMD="${NUMA} ${NUMACTL_ARGS}"
 +else
 +  export NUMA_CMD=""
 +fi
 +
  export HADOOP_HOME=$HADOOP_PREFIX
  export HADOOP_HOME_WARN_SUPPRESS=true
  

http://git-wip-us.apache.org/repos/asf/accumulo/blob/1faee537/assemble/bin/start-server.sh
--
diff --cc assemble/bin/start-server.sh
index 3587ec9,2fb4c4c..8266457
--- a/assemble/bin/start-server.sh
+++ b/assemble/bin/start-server.sh
@@@ -77,12 -77,12 +77,12 @@@ if [[ -z "$PID" ]]; the
COMMAND="${bin}/accumulo_watcher.sh ${LOGHOST}"
 fi
  
 -   if [ "$HOST" = "localhost" -o "$HOST" = "`hostname -f`" -o "$HOST" = "$ip" 
]; then
 -  ${bin}/accumulo ${SERVICE} --address ${ADDRESS} 
>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err & 
 +   if [[ $HOST == localhost || $HOST == "$(hostname -f)" || $HOST = "$IP" ]]; 
then
-   ${NUMA_CMD} "$COMMAND" "${SERVICE}" --address "${ADDRESS}" 
>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out" 
2>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err" & 
++  ${NUMA_CMD} "$COMMAND" "${SERVICE}" --address "${ADDRESS}" 
>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out" 
2>"${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err" &
MAX_FILES_OPEN=$(ulimit -n)
 else
 -  $SSH $HOST "bash -c 'exec nohup ${bin}/accumulo ${SERVICE} --address 
${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
 -  MAX_FILES_OPEN=$($SSH $HOST "/usr/bin/env bash -c 'ulimit -n'") 
 +  $SSH "$HOST" "bash -c 'exec nohup ${NUMA_CMD} $COMMAND ${SERVICE} 
--address ${ADDRESS} >${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.out 
2>${ACCUMULO_LOG_DIR}/${SERVICE}_${LOGHOST}.err' &"
-   MAX_FILES_OPEN=$($SSH "$HOST" "/usr/bin/env bash -c 'ulimit -n'") 
++  MAX_FILES_OPEN=$($SSH "$HOST" "/usr/bin/env bash -c 'ulimit -n'")
 fi
  
 if [[ -n $MAX_FILES_OPEN && -n $SLAVES ]] ; then

http://git-wip-us.apache.org/repos/asf/accumulo/blob/1faee537/pom.xml
--
diff --cc pom.xml
index 19ff679,24f25c9..3815ad7
--- a/pom.xml
+++ b/pom.xml
@@@ -95,7 -95,7 +95,7 @@@

  scm:git:git://git.apache.org/accumulo.git
  
scm:git:https://git-wip-us.apache.org/repos/asf/accumulo.git
- ${project.version}
 -1.7.0
++HEAD
  https://git-wip-us.apache.org/repos/asf?p=accumulo.git





[12/21] accumulo git commit: ACCUMULO-3743 One more CHANGES change.

2015-05-20 Thread ctubbsii
ACCUMULO-3743 One more CHANGES change.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/9fda3bbb
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/9fda3bbb
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/9fda3bbb

Branch: refs/heads/master
Commit: 9fda3bbb2c5a06764412c2ed665e47e75a9762da
Parents: 0d9c05e
Author: Josh Elser 
Authored: Tue May 12 13:42:23 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 13:42:23 2015 -0400

--
 CHANGES | 1 +
 1 file changed, 1 insertion(+)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/9fda3bbb/CHANGES
--
diff --git a/CHANGES b/CHANGES
index 7750d66..d966db4 100644
--- a/CHANGES
+++ b/CHANGES
@@ -474,6 +474,7 @@ Release Notes - Accumulo - Version 1.7.0
 * [ACCUMULO-3791] - Agitator start/stop scripts collide variable names
 * [ACCUMULO-3792] - 
org.apache.accumulo.test.randomwalk.Node#getMapReduceJars needs to include 
htrace
 * [ACCUMULO-3800] - Extra DEPENDENCIES file after new apache.pom
+* [ACCUMULO-3803] - Resurrect mapreduce.lib.util for 1.7.0
 
 ** Improvement
 * [ACCUMULO-898] - look into replacing cloudtrace



[06/21] accumulo git commit: Revert "ACCUMULO-3800 Add exclusion for DEPENDENCIES"

2015-05-20 Thread ctubbsii
Revert "ACCUMULO-3800 Add exclusion for DEPENDENCIES"

This reverts commit 8e99ed222db49e0f9a4f03ae90d1cbcf44bb1a87.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/dc1d0def
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/dc1d0def
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/dc1d0def

Branch: refs/heads/master
Commit: dc1d0defb79b5fde68631f1063be021fb3420f74
Parents: c16fd9d
Author: Christopher Tubbs 
Authored: Tue May 12 11:42:12 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 12:44:01 2015 -0400

--
 pom.xml | 9 -
 1 file changed, 9 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/dc1d0def/pom.xml
--
diff --git a/pom.xml b/pom.xml
index f4c734b..5bf65e8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -955,15 +955,6 @@
 
   
 
-
-  org.apache.rat
-  apache-rat-plugin
-  
-
-  DEPENDENCIES
-
-  
-
   
 
 



[13/21] accumulo git commit: [maven-release-plugin] prepare release 1.7.0

2015-05-20 Thread ctubbsii
[maven-release-plugin] prepare release 1.7.0


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/8cba8128
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/8cba8128
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/8cba8128

Branch: refs/heads/1.7
Commit: 8cba8128fbc3238bdd9398cf5c36b7cb6dc3b61d
Parents: 9fda3bb
Author: Josh Elser 
Authored: Tue May 12 14:34:29 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 14:34:29 2015 -0400

--
 assemble/pom.xml| 2 +-
 core/pom.xml| 2 +-
 docs/pom.xml| 2 +-
 examples/simple/pom.xml | 2 +-
 fate/pom.xml| 2 +-
 maven-plugin/pom.xml| 2 +-
 minicluster/pom.xml | 2 +-
 pom.xml | 4 ++--
 proxy/pom.xml   | 2 +-
 server/base/pom.xml | 2 +-
 server/gc/pom.xml   | 2 +-
 server/master/pom.xml   | 2 +-
 server/monitor/pom.xml  | 2 +-
 server/native/pom.xml   | 2 +-
 server/tracer/pom.xml   | 2 +-
 server/tserver/pom.xml  | 2 +-
 shell/pom.xml   | 2 +-
 start/pom.xml   | 2 +-
 test/pom.xml| 2 +-
 trace/pom.xml   | 2 +-
 20 files changed, 21 insertions(+), 21 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/assemble/pom.xml
--
diff --git a/assemble/pom.xml b/assemble/pom.xml
index e88cd81..0cf690b 100644
--- a/assemble/pom.xml
+++ b/assemble/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo
   pom

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/core/pom.xml
--
diff --git a/core/pom.xml b/core/pom.xml
index 12e3c1d..fe91d0a 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-core
   Core

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/docs/pom.xml
--
diff --git a/docs/pom.xml b/docs/pom.xml
index 47d98da..519f534 100644
--- a/docs/pom.xml
+++ b/docs/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-docs
   pom

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/examples/simple/pom.xml
--
diff --git a/examples/simple/pom.xml b/examples/simple/pom.xml
index aa2f118..7172831 100644
--- a/examples/simple/pom.xml
+++ b/examples/simple/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
 ../../pom.xml
   
   accumulo-examples-simple

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/fate/pom.xml
--
diff --git a/fate/pom.xml b/fate/pom.xml
index 5fd9a7e..e282a14 100644
--- a/fate/pom.xml
+++ b/fate/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-fate
   Fate

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/maven-plugin/pom.xml
--
diff --git a/maven-plugin/pom.xml b/maven-plugin/pom.xml
index 2c07d7d..ecb3bd0 100644
--- a/maven-plugin/pom.xml
+++ b/maven-plugin/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-maven-plugin
   maven-plugin

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/minicluster/pom.xml
--
diff --git a/minicluster/pom.xml b/minicluster/pom.xml
index 7644269..c12c967 100644
--- a/minicluster/pom.xml
+++ b/minicluster/pom.xml
@@ -20,7 +20,7 @@
   
 org.apache.accumulo
 accumulo-project
-1.7.0-SNAPSHOT
+1.7.0
   
   accumulo-minicluster
   MiniCluster

http://git-wip-us.apache.org/repos/asf/accumulo/blob/8cba8128/pom.xml
--
diff --git a/pom.xml b/pom.xml
index 5bf65e8..24f25c9 100644
--- a/pom.xml
+++ b/pom.xml
@@ -24,7 +24,7 @@
   
   org.apache.accumulo
   accumulo-project
-  1.7.0-SNAPSHOT
+  1.7.0
   pom
   Apache Accumulo
   Apache Accumulo is a sorted, distributed key/value store based 
on Google's BigTable design. It is built on top of Apache Hadoop, Zookeeper, 
and Thrift. It features a few novel improvements on the BigTable design in the 
form of cell-level access labels and a server-side programming mechanism that 
can modify key/value pairs at various points in the data management 
process.
@@ -95

[10/21] accumulo git commit: ACCUMULO-3803 Resurected mapreduce.lib.util

2015-05-20 Thread ctubbsii
ACCUMULO-3803 Resurected mapreduce.lib.util

Ran the following command in the 1.7 branch to do this.  The commit below is 
what was at head of the 1.6 branch at the time I ran this command.

git checkout c4eff0c2eb1320e411ac3e41b6f2db89c2d3ba33 -- 
core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util

I checked in the 1.6 branch and found that this code has no references (like 
test).


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/0d9c05e7
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/0d9c05e7
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/0d9c05e7

Branch: refs/heads/master
Commit: 0d9c05e716774f7904c7da2664063a778ab4640b
Parents: eef4dfe
Author: Keith Turner 
Authored: Tue May 12 12:51:18 2015 -0400
Committer: Josh Elser 
Committed: Tue May 12 13:40:37 2015 -0400

--
 .../mapreduce/lib/util/ConfiguratorBase.java| 277 +++
 .../lib/util/FileOutputConfigurator.java| 170 +++
 .../mapreduce/lib/util/InputConfigurator.java   | 462 +++
 .../mapreduce/lib/util/OutputConfigurator.java  | 196 
 .../client/mapreduce/lib/util/package-info.java |  22 +
 5 files changed, 1127 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/accumulo/blob/0d9c05e7/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
--
diff --git 
a/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
 
b/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
new file mode 100644
index 000..20fbbea
--- /dev/null
+++ 
b/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
@@ -0,0 +1,277 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.accumulo.core.client.mapreduce.lib.util;
+
+import org.apache.accumulo.core.client.AccumuloSecurityException;
+import org.apache.accumulo.core.client.ClientConfiguration;
+import org.apache.accumulo.core.client.Instance;
+import org.apache.accumulo.core.client.ZooKeeperInstance;
+import org.apache.accumulo.core.client.mock.MockInstance;
+import org.apache.accumulo.core.client.security.tokens.AuthenticationToken;
+import 
org.apache.accumulo.core.client.security.tokens.AuthenticationToken.AuthenticationTokenSerializer;
+import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.util.StringUtils;
+import org.apache.log4j.Level;
+
+/**
+ * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+ * @since 1.5.0
+ */
+@Deprecated
+public class ConfiguratorBase {
+
+  /**
+   * Configuration keys for {@link Instance#getConnector(String, 
AuthenticationToken)}.
+   *
+   * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+   * @since 1.5.0
+   */
+  @Deprecated
+  public static enum ConnectorInfo {
+IS_CONFIGURED, PRINCIPAL, TOKEN, TOKEN_CLASS
+  }
+
+  /**
+   * Configuration keys for {@link Instance}, {@link ZooKeeperInstance}, and 
{@link MockInstance}.
+   *
+   * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+   * @since 1.5.0
+   */
+  @Deprecated
+  protected static enum InstanceOpts {
+TYPE, NAME, ZOO_KEEPERS;
+  }
+
+  /**
+   * Configuration keys for general configuration options.
+   *
+   * @deprecated since 1.6.0; Configure your job with the appropriate 
InputFormat or OutputFormat.
+   * @since 1.5.0
+   */
+  @Deprecated
+  protected static enum GeneralOpts {
+LOG_LEVEL
+  }
+
+  /**
+   * Provides a configuration key for a given feature enum, prefixed by the 
implementingClass
+   *
+   * @param implementingClass
+   *  the class whose name will be used as a prefix for the property 
configuration key
+   * @param e
+   *  the enum used to provide the unique part of the configuration key
+   * @return the configuration key
+   * @deprec

svn commit: r1680599 - /accumulo/site/trunk/content/release_notes/1.7.0.mdtext

2015-05-20 Thread ctubbsii
Author: ctubbsii
Date: Wed May 20 15:17:49 2015
New Revision: 1680599

URL: http://svn.apache.org/r1680599
Log:
Minor punctuation, typo fix in new paragraph.

Modified:
accumulo/site/trunk/content/release_notes/1.7.0.mdtext

Modified: accumulo/site/trunk/content/release_notes/1.7.0.mdtext
URL: 
http://svn.apache.org/viewvc/accumulo/site/trunk/content/release_notes/1.7.0.mdtext?rev=1680599&r1=1680598&r2=1680599&view=diff
==
--- accumulo/site/trunk/content/release_notes/1.7.0.mdtext (original)
+++ accumulo/site/trunk/content/release_notes/1.7.0.mdtext Wed May 20 15:17:49 
2015
@@ -310,10 +310,10 @@ subset of all tables.
 
 ## Table and namespace custom properties ##
 
-Inorder to avoid errors caused by mis-typed configuration properties, Accumulo 
was strict about which configuration properties 
-could be set.  However this prevented user from setting arbitrary properties 
that could be used by custom balancers, compaction 
-strategies, volume choosers, and iterators.  Under 
[ACCUMULO-2841][ACCUMULO-2841] the ability to set arbitrary table and 
-namespace properties was added.  The properties need to be prefixed with 
`table.custom`.  The changes made in 
+In order to avoid errors caused by mis-typed configuration properties, 
Accumulo was strict about which configuration properties 
+could be set. However, this prevented users from setting arbitrary properties 
that could be used by custom balancers, compaction 
+strategies, volume choosers, and iterators. Under 
[ACCUMULO-2841][ACCUMULO-2841], the ability to set arbitrary table and 
+namespace properties was added. The properties need to be prefixed with 
`table.custom.`.  The changes made in 
 [ACCUMULO-3177][ACCUMULO-3177] and [ACCUMULO-3439][ACCUMULO-3439] leverage 
this new feature.
 
 # Notable Bug Fixes #




svn commit: r952049 - in /websites/staging/accumulo/trunk/content: ./ release_notes/1.7.0.html

2015-05-20 Thread buildbot
Author: buildbot
Date: Wed May 20 15:17:55 2015
New Revision: 952049

Log:
Staging update by buildbot for accumulo

Modified:
websites/staging/accumulo/trunk/content/   (props changed)
websites/staging/accumulo/trunk/content/release_notes/1.7.0.html

Propchange: websites/staging/accumulo/trunk/content/
--
--- cms:source-revision (original)
+++ cms:source-revision Wed May 20 15:17:55 2015
@@ -1 +1 @@
-1680588
+1680599

Modified: websites/staging/accumulo/trunk/content/release_notes/1.7.0.html
==
--- websites/staging/accumulo/trunk/content/release_notes/1.7.0.html (original)
+++ websites/staging/accumulo/trunk/content/release_notes/1.7.0.html Wed May 20 
15:17:55 2015
@@ -456,10 +456,10 @@ introduces a new per-table property whic
 utilization happens when multiple are available is limited to the expected
 subset of all tables.
 Table and namespace custom 
properties
-Inorder to avoid errors caused by mis-typed configuration properties, 
Accumulo was strict about which configuration properties 
-could be set.  However this prevented user from setting arbitrary properties 
that could be used by custom balancers, compaction 
-strategies, volume choosers, and iterators.  Under https://issues.apache.org/jira/browse/ACCUMULO-2841";>ACCUMULO-2841 
the ability to set arbitrary table and 
-namespace properties was added.  The properties need to be prefixed with 
table.custom.  The changes made in 
+In order to avoid errors caused by mis-typed configuration properties, 
Accumulo was strict about which configuration properties 
+could be set. However, this prevented users from setting arbitrary properties 
that could be used by custom balancers, compaction 
+strategies, volume choosers, and iterators. Under https://issues.apache.org/jira/browse/ACCUMULO-2841";>ACCUMULO-2841, 
the ability to set arbitrary table and 
+namespace properties was added. The properties need to be prefixed with 
table.custom..  The changes made in 
 https://issues.apache.org/jira/browse/ACCUMULO-3177";>ACCUMULO-3177 
and https://issues.apache.org/jira/browse/ACCUMULO-3439";>ACCUMULO-3439 
leverage this new feature.
 Notable Bug Fixes
 SourceSwitchingIterator Deadlock




svn commit: r952047 - in /websites/production/accumulo/content: ./ 1.4/apidocs/ 1.5/apidocs/ 1.6/apidocs/ 1.7/apidocs/

2015-05-20 Thread kturner
Author: kturner
Date: Wed May 20 14:58:01 2015
New Revision: 952047

Log:
Publishing svnmucc operation to accumulo site by kturner

Added:
websites/production/accumulo/content/
  - copied from r952046, websites/staging/accumulo/trunk/content/
websites/production/accumulo/content/1.4/apidocs/
  - copied from r952046, websites/production/accumulo/content/1.4/apidocs/
websites/production/accumulo/content/1.5/apidocs/
  - copied from r952046, websites/production/accumulo/content/1.5/apidocs/
websites/production/accumulo/content/1.6/apidocs/
  - copied from r952046, websites/production/accumulo/content/1.6/apidocs/
websites/production/accumulo/content/1.7/apidocs/
  - copied from r952046, websites/production/accumulo/content/1.7/apidocs/



svn commit: r952046 - in /websites/staging/accumulo/trunk/content: ./ release_notes/1.7.0.html

2015-05-20 Thread buildbot
Author: buildbot
Date: Wed May 20 14:57:06 2015
New Revision: 952046

Log:
Staging update by buildbot for accumulo

Modified:
websites/staging/accumulo/trunk/content/   (props changed)
websites/staging/accumulo/trunk/content/release_notes/1.7.0.html

Propchange: websites/staging/accumulo/trunk/content/
--
--- cms:source-revision (original)
+++ cms:source-revision Wed May 20 14:57:06 2015
@@ -1 +1 @@
-1680587
+1680588

Modified: websites/staging/accumulo/trunk/content/release_notes/1.7.0.html
==
--- websites/staging/accumulo/trunk/content/release_notes/1.7.0.html (original)
+++ websites/staging/accumulo/trunk/content/release_notes/1.7.0.html Wed May 20 
14:57:06 2015
@@ -455,7 +455,7 @@ introduces a new per-table property whic
 VolumeChooser. This ensures that the implementation to choose how 
HDFS
 utilization happens when multiple are available is limited to the expected
 subset of all tables.
-Table and namespace custom 
properties
+Table and namespace custom 
properties
 Inorder to avoid errors caused by mis-typed configuration properties, 
Accumulo was strict about which configuration properties 
 could be set.  However this prevented user from setting arbitrary properties 
that could be used by custom balancers, compaction 
 strategies, volume choosers, and iterators.  Under https://issues.apache.org/jira/browse/ACCUMULO-2841";>ACCUMULO-2841 
the ability to set arbitrary table and 




svn commit: r1680588 - /accumulo/site/trunk/content/release_notes/1.7.0.mdtext

2015-05-20 Thread kturner
Author: kturner
Date: Wed May 20 14:57:00 2015
New Revision: 1680588

URL: http://svn.apache.org/r1680588
Log:
ACCUMULO-3737 Fix header

Modified:
accumulo/site/trunk/content/release_notes/1.7.0.mdtext

Modified: accumulo/site/trunk/content/release_notes/1.7.0.mdtext
URL: 
http://svn.apache.org/viewvc/accumulo/site/trunk/content/release_notes/1.7.0.mdtext?rev=1680588&r1=1680587&r2=1680588&view=diff
==
--- accumulo/site/trunk/content/release_notes/1.7.0.mdtext (original)
+++ accumulo/site/trunk/content/release_notes/1.7.0.mdtext Wed May 20 14:57:00 
2015
@@ -308,7 +308,7 @@ introduces a new per-table property whic
 utilization happens when multiple are available is limited to the expected
 subset of all tables.
 
-### Table and namespace custom properties
+## Table and namespace custom properties ##
 
 Inorder to avoid errors caused by mis-typed configuration properties, Accumulo 
was strict about which configuration properties 
 could be set.  However this prevented user from setting arbitrary properties 
that could be used by custom balancers, compaction 




svn commit: r1680587 - /accumulo/site/trunk/content/release_notes/1.7.0.mdtext

2015-05-20 Thread kturner
Author: kturner
Date: Wed May 20 14:55:42 2015
New Revision: 1680587

URL: http://svn.apache.org/r1680587
Log:
ACCUMULO-3737 added info about ACCUMULO-2841 to 1.7.0 release notes.

Modified:
accumulo/site/trunk/content/release_notes/1.7.0.mdtext

Modified: accumulo/site/trunk/content/release_notes/1.7.0.mdtext
URL: 
http://svn.apache.org/viewvc/accumulo/site/trunk/content/release_notes/1.7.0.mdtext?rev=1680587&r1=1680586&r2=1680587&view=diff
==
--- accumulo/site/trunk/content/release_notes/1.7.0.mdtext (original)
+++ accumulo/site/trunk/content/release_notes/1.7.0.mdtext Wed May 20 14:55:42 
2015
@@ -308,6 +308,14 @@ introduces a new per-table property whic
 utilization happens when multiple are available is limited to the expected
 subset of all tables.
 
+### Table and namespace custom properties
+
+Inorder to avoid errors caused by mis-typed configuration properties, Accumulo 
was strict about which configuration properties 
+could be set.  However this prevented user from setting arbitrary properties 
that could be used by custom balancers, compaction 
+strategies, volume choosers, and iterators.  Under 
[ACCUMULO-2841][ACCUMULO-2841] the ability to set arbitrary table and 
+namespace properties was added.  The properties need to be prefixed with 
`table.custom`.  The changes made in 
+[ACCUMULO-3177][ACCUMULO-3177] and [ACCUMULO-3439][ACCUMULO-3439] leverage 
this new feature.
+
 # Notable Bug Fixes #
 
 ## SourceSwitchingIterator Deadlock ##
@@ -418,6 +426,7 @@ Accumulo configuration from `120s` to `2
 [ACCUMULO-1957]: https://issues.apache.org/jira/browse/ACCUMULO-1957
 [ACCUMULO-2388]: https://issues.apache.org/jira/browse/ACCUMULO-2388
 [ACCUMULO-2815]: https://issues.apache.org/jira/browse/ACCUMULO-2815
+[ACCUMULO-2841]: https://issues.apache.org/jira/browse/ACCUMULO-2841
 [ACCUMULO-2863]: https://issues.apache.org/jira/browse/ACCUMULO-2863
 [ACCUMULO-2998]: https://issues.apache.org/jira/browse/ACCUMULO-2998
 [ACCUMULO-3134]: https://issues.apache.org/jira/browse/ACCUMULO-3134




svn commit: r952045 - in /websites/staging/accumulo/trunk/content: ./ release_notes/1.7.0.html

2015-05-20 Thread buildbot
Author: buildbot
Date: Wed May 20 14:55:51 2015
New Revision: 952045

Log:
Staging update by buildbot for accumulo

Modified:
websites/staging/accumulo/trunk/content/   (props changed)
websites/staging/accumulo/trunk/content/release_notes/1.7.0.html

Propchange: websites/staging/accumulo/trunk/content/
--
--- cms:source-revision (original)
+++ cms:source-revision Wed May 20 14:55:51 2015
@@ -1 +1 @@
-1680535
+1680587

Modified: websites/staging/accumulo/trunk/content/release_notes/1.7.0.html
==
--- websites/staging/accumulo/trunk/content/release_notes/1.7.0.html (original)
+++ websites/staging/accumulo/trunk/content/release_notes/1.7.0.html Wed May 20 
14:55:51 2015
@@ -455,6 +455,12 @@ introduces a new per-table property whic
 VolumeChooser. This ensures that the implementation to choose how 
HDFS
 utilization happens when multiple are available is limited to the expected
 subset of all tables.
+Table and namespace custom 
properties
+Inorder to avoid errors caused by mis-typed configuration properties, 
Accumulo was strict about which configuration properties 
+could be set.  However this prevented user from setting arbitrary properties 
that could be used by custom balancers, compaction 
+strategies, volume choosers, and iterators.  Under https://issues.apache.org/jira/browse/ACCUMULO-2841";>ACCUMULO-2841 
the ability to set arbitrary table and 
+namespace properties was added.  The properties need to be prefixed with 
table.custom.  The changes made in 
+https://issues.apache.org/jira/browse/ACCUMULO-3177";>ACCUMULO-3177 
and https://issues.apache.org/jira/browse/ACCUMULO-3439";>ACCUMULO-3439 
leverage this new feature.
 Notable Bug Fixes
 SourceSwitchingIterator Deadlock
 An instance of SourceSwitchingIterator, the Accumulo iterator which




svn commit: r952024 - in /websites/production/accumulo/content: ./ 1.4/apidocs/ 1.5/apidocs/ 1.6/apidocs/ 1.7/apidocs/

2015-05-20 Thread ecn
Author: ecn
Date: Wed May 20 12:10:29 2015
New Revision: 952024

Log:
Publishing svnmucc operation to accumulo site by ecn

Added:
websites/production/accumulo/content/
  - copied from r952023, websites/staging/accumulo/trunk/content/
websites/production/accumulo/content/1.4/apidocs/
  - copied from r952023, websites/production/accumulo/content/1.4/apidocs/
websites/production/accumulo/content/1.5/apidocs/
  - copied from r952023, websites/production/accumulo/content/1.5/apidocs/
websites/production/accumulo/content/1.6/apidocs/
  - copied from r952023, websites/production/accumulo/content/1.6/apidocs/
websites/production/accumulo/content/1.7/apidocs/
  - copied from r952023, websites/production/accumulo/content/1.7/apidocs/



svn commit: r952022 - in /websites/staging/accumulo/trunk/content: ./ release_notes/1.7.0.html

2015-05-20 Thread buildbot
Author: buildbot
Date: Wed May 20 12:10:02 2015
New Revision: 952022

Log:
Staging update by buildbot for accumulo

Modified:
websites/staging/accumulo/trunk/content/   (props changed)
websites/staging/accumulo/trunk/content/release_notes/1.7.0.html

Propchange: websites/staging/accumulo/trunk/content/
--
--- cms:source-revision (original)
+++ cms:source-revision Wed May 20 12:10:02 2015
@@ -1 +1 @@
-1680454
+1680535

Modified: websites/staging/accumulo/trunk/content/release_notes/1.7.0.html
==
--- websites/staging/accumulo/trunk/content/release_notes/1.7.0.html (original)
+++ websites/staging/accumulo/trunk/content/release_notes/1.7.0.html Wed May 20 
12:10:02 2015
@@ -343,19 +343,12 @@ using Accumulo's public API. The regular
 https://github.com/apache/accumulo/blob/1.7.0/README.md#api";>README.
 Performance Improvements
 Configurable Threadpool 
Size for Assignments
-One of the primary tasks that the Accumulo Master is responsible for is the
-assignment of tablets to tablet servers. Before a tablet can be brought
-online, it must not have any outstanding logs because this represents a need
-to perform recovery (the tablet was not unloaded cleanly). This process can
-take some time for large write-ahead log files, so it is performed on a tablet
-server to keep the Master light and agile.
-Assignments, whether the tablets need to perform recovery or not, share the
-same threadpool in the Master. This means that when a large number of tablet
-servers are available, too few threads dedicated to assignment can restrict
-the speed at which assignments can be performed.
-https://issues.apache.org/jira/browse/ACCUMULO-1085";>ACCUMULO-1085 
allows the size of the threadpool used in the
-Master for assignments to be configurable which can be dynamically altered to
-remove the limitation when sufficient servers are available.
+During start-up, the Master quickly assigns tablets to Tablet Servers. 
However,
+Tablet Servers load those assigned tablets one at a time. In 1.7, the servers
+will be more aggressive, and will load tablets in parallel, so long as they do
+not have mutations that need to be recovered.
+https://issues.apache.org/jira/browse/ACCUMULO-1085";>ACCUMULO-1085 
allows the size of the threadpool used in the Tablet Servers 
+for assignment processing to be configurable.
 Group-Commit 
Threshold as a Factor of Data Size
 When ingesting data into Accumulo, the majority of time is spent in the
 write-ahead log. As such, this is a common place that optimizations are added.




svn commit: r1680535 - /accumulo/site/trunk/content/release_notes/1.7.0.mdtext

2015-05-20 Thread ecn
Author: ecn
Date: Wed May 20 12:09:53 2015
New Revision: 1680535

URL: http://svn.apache.org/r1680535
Log:
CMS commit to accumulo by ecn

Modified:
accumulo/site/trunk/content/release_notes/1.7.0.mdtext

Modified: accumulo/site/trunk/content/release_notes/1.7.0.mdtext
URL: 
http://svn.apache.org/viewvc/accumulo/site/trunk/content/release_notes/1.7.0.mdtext?rev=1680535&r1=1680534&r2=1680535&view=diff
==
--- accumulo/site/trunk/content/release_notes/1.7.0.mdtext (original)
+++ accumulo/site/trunk/content/release_notes/1.7.0.mdtext Wed May 20 12:09:53 
2015
@@ -168,20 +168,13 @@ using Accumulo's public API. The regular
 
 ## Configurable Threadpool Size for Assignments ##
 
-One of the primary tasks that the Accumulo Master is responsible for is the
-assignment of tablets to tablet servers. Before a tablet can be brought
-online, it must not have any outstanding logs because this represents a need
-to perform recovery (the tablet was not unloaded cleanly). This process can
-take some time for large write-ahead log files, so it is performed on a tablet
-server to keep the Master light and agile.
+During start-up, the Master quickly assigns tablets to Tablet Servers. However,
+Tablet Servers load those assigned tablets one at a time. In 1.7, the servers
+will be more aggressive, and will load tablets in parallel, so long as they do
+not have mutations that need to be recovered.
 
-Assignments, whether the tablets need to perform recovery or not, share the
-same threadpool in the Master. This means that when a large number of tablet
-servers are available, too few threads dedicated to assignment can restrict
-the speed at which assignments can be performed.
-[ACCUMULO-1085][ACCUMULO-1085] allows the size of the threadpool used in the
-Master for assignments to be configurable which can be dynamically altered to
-remove the limitation when sufficient servers are available.
+[ACCUMULO-1085] allows the size of the threadpool used in the Tablet Servers 
+for assignment processing to be configurable.
 
 ## Group-Commit Threshold as a Factor of Data Size ##