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

Review request for hcatalog, Dmitriy Ryaboy and Feng Peng.


Description
-------

HCatalog should use checkstyle to enforce coding style


This addresses bug HCATALOG-486.
    https://issues.apache.org/jira/browse/HCATALOG-486


Diffs
-----

  ant/checkstyle.xml PRE-CREATION 
  ant/deploy.xml 7e2934c 
  build.xml 37af4a7 
  coding_style.xml PRE-CREATION 
  hcatalog-pig-adapter/src/main/java/org/apache/hcatalog/pig/HCatLoader.java 
579811a 
  hcatalog-pig-adapter/src/main/java/org/apache/hcatalog/pig/HCatStorer.java 
7696587 
  
hcatalog-pig-adapter/src/main/java/org/apache/hcatalog/pig/drivers/LoadFuncBasedInputFormat.java
 c6ecae3 
  
hcatalog-pig-adapter/src/main/java/org/apache/hcatalog/pig/drivers/StoreFuncBasedOutputFormat.java
 67a4ccd 
  ivy.xml 0230b5c 
  ivy/libraries.properties 4d199f3 
  pom.xml PRE-CREATION 
  shims/src/23/java/org/apache/hcatalog/shims/HCatHadoopShims23.java e7bf16a 
  src/java/org/apache/hcatalog/data/DefaultHCatRecord.java 5606c54 
  src/java/org/apache/hcatalog/data/HCatRecord.java 9404a7d 
  src/java/org/apache/hcatalog/data/HCatRecordable.java 68d8d8b 
  src/java/org/apache/hcatalog/data/transfer/ReadEntity.java 45e01d9 
  src/java/org/apache/hcatalog/data/transfer/impl/HCatInputFormatReader.java 
3580612 
  src/java/org/apache/hcatalog/data/transfer/impl/HCatOutputFormatWriter.java 
c30ca36 
  src/java/org/apache/hcatalog/data/transfer/state/StateProvider.java 4b09a59 
  src/java/org/apache/hcatalog/listener/NotificationListener.java 63775f7 
  src/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java 
e9f3fa3 
  src/java/org/apache/hcatalog/mapreduce/HCatBaseOutputFormat.java e76690f 
  src/java/org/apache/hcatalog/mapreduce/MultiOutputFormat.java 41d458f 
  src/java/org/apache/hcatalog/mapreduce/PartInfo.java 235233f 
  src/java/org/apache/hcatalog/shims/HCatHadoopShims.java e285035 
  
src/test/e2e/hcatalog/tools/generate/java/org/apache/hadoop/hive/tools/generate/RCFileGenerator.java
 a269d44 
  
src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/DataWriterMaster.java 
3dcc1ec 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/GroupByAge.java 
7abaab3 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/HBaseReadWrite.java 
c420233 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/HCatTestDriver.java 
f32a364 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/HCatTypeCheck.java 
0a279f0 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/ReadJson.java 
cc1da1b 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/ReadRC.java 884fa42 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/ReadText.java 
7797ce1 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/StoreDemo.java 
4c97057 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/TypeDataCheck.java 
16f29fb 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/WriteJson.java 
55fff1d 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/WriteRC.java 
83eacfc 
  src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/WriteText.java 
c7293f0 
  
src/test/e2e/hcatalog/udfs/java/org/apache/hcatalog/utils/WriteTextPartitioned.java
 9630ccf 
  src/test/org/apache/hcatalog/ExitException.java 2062f2c 
  src/test/org/apache/hcatalog/NoExitSecurityManager.java 2cc6ff6 
  src/test/org/apache/hcatalog/cli/TestPermsGrp.java 5c094ac 
  src/test/org/apache/hcatalog/common/TestHiveClientCache.java b6241ef 
  src/test/org/apache/hcatalog/data/TestDefaultHCatRecord.java 250c5dc 
  src/test/org/apache/hcatalog/data/TestLazyHCatRecord.java 42c9795 
  src/test/org/apache/hcatalog/data/schema/TestHCatSchema.java 3a72e78 
  src/test/org/apache/hcatalog/mapreduce/HCatMapReduceTest.java 2509c20 
  src/test/org/apache/hcatalog/mapreduce/TestHCatHiveThriftCompatibility.java 
0c5b982 
  src/test/org/apache/hcatalog/mapreduce/TestHCatOutputFormat.java f451082 
  src/test/org/apache/hcatalog/mapreduce/TestPassProperties.java b55ef09 
  storage-handlers/hbase/build.xml 7c06e0b 
  storage-handlers/hbase/ivy.xml 8d1e271 
  storage-handlers/hbase/pom.xml PRE-CREATION 
  
storage-handlers/hbase/src/gen-java/org/apache/hcatalog/hbase/snapshot/transaction/thrift/StoreFamilyRevision.java
 a55dde7 
  
storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/ImportSequenceFile.java
 5f40b42 
  
storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/snapshot/RevisionManager.java
 be07e10 
  
storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/snapshot/RevisionManagerEndpoint.java
 433f5f5 
  
storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/snapshot/RevisionManagerEndpointClient.java
 00db32b 
  
storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/snapshot/RevisionManagerFactory.java
 02e7fc9 
  
storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/snapshot/ZKUtil.java 
0e4066f 
  
storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/snapshot/lock/LockListener.java
 6494279 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/ManyMiniCluster.java 
1c19699 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseDirectOutputFormat.java
 aa2f01b 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseHCatStorageHandler.java
 e6da40b 
  storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/TestSnapshots.java 
b0deb86 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/snapshot/TestRevisionManager.java
 5b63c40 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/snapshot/TestRevisionManagerConfiguration.java
 1fd8a93 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/snapshot/TestRevisionManagerEndpoint.java
 ae93211 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/snapshot/TestThriftSerialization.java
 7414a2a 
  
storage-handlers/hbase/src/test/org/apache/hcatalog/hbase/snapshot/TestZNodeSetUp.java
 aecd2be 
  webhcat/java-client/src/main/java/org/apache/hcatalog/api/HCatClient.java 
91721f5 
  
webhcat/svr/src/main/java/org/apache/hcatalog/templeton/CompleteDelegator.java 
ed0e538 
  webhcat/svr/src/main/java/org/apache/hcatalog/templeton/DeleteDelegator.java 
5e12605 
  
webhcat/svr/src/main/java/org/apache/hcatalog/templeton/GroupPermissionsDesc.java
 b96ef25 
  
webhcat/svr/src/main/java/org/apache/hcatalog/templeton/LauncherDelegator.java 
bdcb1c3 
  webhcat/svr/src/main/java/org/apache/hcatalog/templeton/ListDelegator.java 
60eb7fa 
  webhcat/svr/src/main/java/org/apache/hcatalog/templeton/QueueException.java 
5a9861d 
  
webhcat/svr/src/main/java/org/apache/hcatalog/templeton/SimpleWebException.java 
733bd97 
  
webhcat/svr/src/main/java/org/apache/hcatalog/templeton/tool/TempletonControllerJob.java
 8bdb1f1 
  
webhcat/svr/src/main/java/org/apache/hcatalog/templeton/tool/TempletonUtils.java
 62930b2 
  webhcat/svr/src/test/java/org/apache/hcatalog/templeton/TestDesc.java d13307a 
  webhcat/svr/src/test/java/org/apache/hcatalog/templeton/TestServer.java 
85f35f0 
  
webhcat/svr/src/test/java/org/apache/hcatalog/templeton/tool/TestTempletonUtils.java
 d034a21 
  
webhcat/svr/src/test/java/org/apache/hcatalog/templeton/tool/TestTrivialExecService.java
 9d0af61 

Diff: https://reviews.apache.org/r/6856/diff/


Testing
-------


Thanks,

Travis Crawford

Reply via email to