----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18112/#review34571 -----------------------------------------------------------
I found some concurrency problems in the changes of catalog module. tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStoreClientPool.java <https://reviews.apache.org/r/18112/#comment64699> AtomicBoolean looks enough for this purpose. You don't need to implement this kind of class. tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/MutableBoolean.java <https://reviews.apache.org/r/18112/#comment64700> AtomicBoolean looks enough for this purpose. You don't need to implement this kind of class. tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/CatalogServer.java <https://reviews.apache.org/r/18112/#comment64701> What was the reasoning behind the removal of compareTo function? tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/store/AbstractDBStore.java <https://reviews.apache.org/r/18112/#comment64702> Statement, PreparedStatement, and ResultSet should not be used as member variables. Many methods share them, so this kind of usages definitely causes concurrency problems. - Hyunsik Choi On Feb. 14, 2014, 6:43 p.m., Jung JaeHwa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18112/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2014, 6:43 p.m.) > > > Review request for Tajo. > > > Bugs: TAJO-530 > https://issues.apache.org/jira/browse/TAJO-530 > > > Repository: tajo > > > Description > ------- > > Fix warnings in tajo-catalog. > > > Diffs > ----- > > dev-support/findbugs-exclude.xml 0f967de > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/FunctionDesc.java > 084d97c > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/Options.java > e770b11 > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/Schema.java > 2f57ff2 > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/SortSpec.java > 5cc0de1 > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/TableDesc.java > 1b0a9fe > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/json/TableMetaAdapter.java > ce42bea > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/partition/PartitionDesc.java > f7c1342 > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/partition/PartitionMethodDesc.java > d4a2c3e > > tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/statistics/StatSet.java > f8d24b0 > > tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java > f796fff > > tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStoreClientPool.java > 263e1dd > > tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/MutableBoolean.java > PRE-CREATION > > tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/CatalogServer.java > 62d6e27 > > tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/store/AbstractDBStore.java > 1ac9d80 > > tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/store/DerbyStore.java > 6f1b612 > > tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/store/MySQLStore.java > e9c5a03 > > Diff: https://reviews.apache.org/r/18112/diff/ > > > Testing > ------- > > mvn clean test findbugs:findbugs -Phcatalog-0.12.0 > mvn clean test findbugs:findbugs -Phcatalog-0.11.0 > > > Thanks, > > Jung JaeHwa > >
