[ 
https://issues.apache.org/jira/browse/TAJO-1603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14545220#comment-14545220
 ] 

Hudson commented on TAJO-1603:
------------------------------

FAILURE: Integrated in Tajo-master-CODEGEN-build #348 (See 
[https://builds.apache.org/job/Tajo-master-CODEGEN-build/348/])
TAJO-1603: Refactor StorageManager. (hyunsik) (hyunsik: rev 
5491f0e7507c7efa1b2306d4c1f1d25240e482a9)
* CHANGES
* 
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/TableSpaceManager.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestHashAntiJoinExec.java
* 
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/TableSpace.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestBSTIndexExec.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestMergeScanner.java
* 
tajo-core/src/main/java/org/apache/tajo/master/exec/NonForwardQueryResultFileScanner.java
* tajo-core/src/main/java/org/apache/tajo/querymaster/Stage.java
* tajo-core/src/main/java/org/apache/tajo/master/TajoMaster.java
* 
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/MergeScanner.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/StoreTableExec.java
* tajo-core/src/main/java/org/apache/tajo/worker/TajoWorker.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/SeqScanExec.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestRightOuterMergeJoinExec.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestLineReader.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestHashSemiJoinExec.java
* tajo-core/src/test/java/org/apache/tajo/engine/query/TestHBaseTable.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestFileSystems.java
* tajo-core/src/main/java/org/apache/tajo/querymaster/QueryMasterTask.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestLeftOuterHashJoinExec.java
* 
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/StorageManager.java
* tajo-core/src/test/java/org/apache/tajo/storage/TestRowFile.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/index/TestBSTIndex.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/HashShuffleAppenderManager.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestHashJoinExec.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestMergeJoinExec.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/global/TestBroadcastJoinPlan.java
* tajo-core/src/test/java/org/apache/tajo/master/TestExecutionBlockCursor.java
* 
tajo-storage/tajo-storage-hbase/src/main/java/org/apache/tajo/storage/hbase/HBasePutAppender.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestPhysicalPlanner.java
* tajo-core/src/main/java/org/apache/tajo/master/exec/DDLExecutor.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestDelimitedTextFile.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/RangeShuffleFileWriteExec.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestRightOuterHashJoinExec.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestFullOuterHashJoinExec.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestProgressExternalSortExec.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestNLJoinExec.java
* tajo-core/src/main/java/org/apache/tajo/querymaster/Query.java
* 
tajo-storage/tajo-storage-hbase/src/main/java/org/apache/tajo/storage/hbase/HBaseScanner.java
* tajo-core/src/main/java/org/apache/tajo/master/exec/QueryExecutor.java
* tajo-core/src/main/java/org/apache/tajo/querymaster/Repartitioner.java
* tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java
* 
tajo-storage/tajo-storage-hbase/src/main/java/org/apache/tajo/storage/hbase/HBaseStorageManager.java
* tajo-core/src/test/java/org/apache/tajo/engine/query/TestJoinBroadcast.java
* tajo-core/src/main/java/org/apache/tajo/worker/Task.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/json/TestJsonSerDe.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/BSTIndexScanExec.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/FileStorageManager.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestSortExec.java
* tajo-core/src/test/java/org/apache/tajo/jdbc/TestResultSet.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/ColPartitionStoreExec.java
* 
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/FileAppender.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestExternalSortExec.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestStorages.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestBNLJoinExec.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestFileStorageManager.java
* tajo-core/src/test/java/org/apache/tajo/BackendTestingUtil.java
* 
tajo-storage/tajo-storage-hbase/src/test/java/org/apache/tajo/storage/hbase/TestHBaseStorageManager.java
* 
tajo-core/src/main/java/org/apache/tajo/engine/planner/PhysicalPlannerImpl.java
* 
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestFullOuterMergeJoinExec.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/index/TestSingleCSVFileBSTIndex.java
* 
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestCompressionStorages.java


> Refactor StorageManager
> -----------------------
>
>                 Key: TAJO-1603
>                 URL: https://issues.apache.org/jira/browse/TAJO-1603
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: storage
>            Reporter: Hyunsik Choi
>            Assignee: Hyunsik Choi
>            Priority: Minor
>             Fix For: 0.11.0
>
>         Attachments: TAJO-1603.patch
>
>
> This patch refactors StorageManager. Especially, it apparently separates 
> multiple roles into different classes. 
> * Static methods which handle registered store types are moved to 
> TableSpaceManager. 
> * All methods related for file store types are moved to FileStorageManager.
> * I created TableSpace, an interface for tablespace and I modified 
> StorageManager to inherit TableSpace.
>  * Later, StorageManager will be changed to AbstractTableSpace.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to