Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo >> mail # dev >> Review Request 20636: ACCUMULO-2726 restore binary compatibility 1.5 -> 1.6


Copy link to this message
-
Re: Review Request 20636: ACCUMULO-2726 restore binary compatibility 1.5 -> 1.6

This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20636/

(Updated April 25, 2014, 7:19 a.m.)
Review request for accumulo.
Changes

cleaned up whitespace, unused imports, annotations, and tags. made warnings around things that weren't supposed to be public api stronger. moved more out of public api.

ACCUMULO-2733 consolidates manual deserializations into a fromThrift call per discussion on IRC. If that ticket gets +1ed before this one does, I'll refactor this patch to be based on that one.
Bugs: ACCUMULO-2726
    https://issues.apache.org/jira/browse/ACCUMULO-2726
Repository: accumulo
Description

restores things found missing by japi compliance checker
Diffs (updated)

  core/src/main/java/org/apache/accumulo/core/client/admin/ActiveCompaction.java 9c39ea6
  core/src/main/java/org/apache/accumulo/core/client/admin/ActiveScan.java 30e47af
  core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperationsImpl.java f80eee5
  core/src/main/java/org/apache/accumulo/core/client/admin/NamespaceOperationsHelper.java b9a7791
  core/src/main/java/org/apache/accumulo/core/client/admin/NamespaceOperationsImpl.java 569a3b6
  core/src/main/java/org/apache/accumulo/core/client/admin/SecurityOperationsImpl.java 9d662f4
  core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsHelper.java 843f572
  core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsImpl.java 3d69cc1
  core/src/main/java/org/apache/accumulo/core/client/impl/ActiveCompactionImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/impl/ActiveScanImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/impl/ConnectorImpl.java 7d9d3ab
  core/src/main/java/org/apache/accumulo/core/client/impl/InstanceOperationsImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/impl/NamespaceOperationsHelper.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/impl/NamespaceOperationsImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/impl/SecurityOperationsImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/impl/TableOperationsHelper.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/impl/TableOperationsImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/impl/InputConfigurator.java cfd9aa2
  core/src/main/java/org/apache/accumulo/core/client/mock/MockConnector.java 996198c
  core/src/main/java/org/apache/accumulo/core/client/mock/MockInstanceOperations.java 15379af
  core/src/main/java/org/apache/accumulo/core/client/mock/MockInstanceOperationsImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/mock/MockNamespaceOperations.java 9f0594a
  core/src/main/java/org/apache/accumulo/core/client/mock/MockSecurityOperations.java 16a8e02
  core/src/main/java/org/apache/accumulo/core/client/mock/MockSecurityOperationsImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/mock/MockShell.java 2bc9436
  core/src/main/java/org/apache/accumulo/core/client/mock/MockTableOperations.java d3b1571
  core/src/main/java/org/apache/accumulo/core/client/mock/MockTableOperationsImpl.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/client/mock/MockTabletLocator.java 6bd01a9
  core/src/main/java/org/apache/accumulo/core/client/mock/impl/MockTabletLocator.java PRE-CREATION
  core/src/main/java/org/apache/accumulo/core/security/Credentials.java 5afc6e8
  core/src/test/java/org/apache/accumulo/core/client/admin/TableOperationsHelperTest.java 32136a8
  core/src/test/java/org/apache/accumulo/core/client/impl/TableOperationsHelperTest.java PRE-CREATION
  core/src/test/java/org/apache/accumulo/core/security/CredentialsTest.java 4f8079e
  server/base/src/main/java/org/apache/accumulo/server/security/SecurityOperation.java c2a7001
  server/master/src/main/java/org/apache/accumulo/master/FateServiceHandler.java d63a63e
  server/master/src/main/java/org/apache/accumulo/master/tableOps/ImportTable.java cd59b78
  test/compat/japi-compliance/japi-accumulo-1.5.xml 9e6f47f
  test/compat/japi-compliance/japi-accumulo-1.6.xml 36553b8

Diff: https://reviews.apache.org/r/20636/diff/
Testing

unit tests pass. ITs passed.
Thanks,

Sean Busbey