[22/49] carbondata git commit: [CARBONDATA-2025] Unify all path construction through CarbonTablePath static method

2018-02-28 Thread jackylk
[CARBONDATA-2025] Unify all path construction through CarbonTablePath static 
method

Refactory CarbonTablePath:

1.Remove CarbonStorePath and use CarbonTablePath only.
2.Make CarbonTablePath an utility without object creation, it can avoid 
creating object before using it, thus code is cleaner and GC is less.

This closes #1768


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

Branch: refs/heads/carbonstore-rebase5
Commit: eec8f18ac3c0715671146ddf4151bf7b4ca54f46
Parents: 7e37843
Author: Jacky Li 
Authored: Wed Jan 31 16:14:27 2018 +0800
Committer: Jacky Li 
Committed: Wed Feb 28 22:05:18 2018 +0800

--
 .../DictionaryColumnUniqueIdentifier.java   |  29 +-
 .../dictionary/ManageDictionaryAndBTree.java|  13 +-
 .../core/metadata/AbsoluteTableIdentifier.java  |   4 +-
 .../core/metadata/schema/table/CarbonTable.java |  11 +-
 .../core/mutate/CarbonUpdateUtil.java   |  58 ++--
 .../core/scan/executor/util/QueryUtil.java  |   7 +-
 .../core/service/CarbonCommonFactory.java   |  16 -
 .../carbondata/core/service/PathService.java|  35 ---
 .../core/service/impl/PathFactory.java  |  50 
 .../statusmanager/SegmentStatusManager.java |  30 +-
 .../SegmentUpdateStatusManager.java |  59 ++--
 .../apache/carbondata/core/util/CarbonUtil.java |  60 ++--
 .../util/path/CarbonSharedDictionaryPath.java   |  71 -
 .../core/util/path/CarbonStorePath.java |  71 -
 .../core/util/path/CarbonTablePath.java | 291 ++-
 .../dictionary/AbstractDictionaryCacheTest.java |  11 +-
 .../dictionary/ForwardDictionaryCacheTest.java  |   6 +-
 .../dictionary/ReverseDictionaryCacheTest.java  |   6 +-
 .../reader/CarbonDictionaryReaderImplTest.java  |   8 -
 .../CarbonFormatDirectoryStructureTest.java |  18 +-
 .../path/CarbonFormatSharedDictionaryTest.java  |  44 ---
 .../writer/CarbonDictionaryWriterImplTest.java  |  19 +-
 .../CarbonBatchSparkStreamingExample.scala  |   9 +-
 .../CarbonStructuredStreamingExample.scala  |  11 +-
 .../hadoop/api/CarbonTableInputFormat.java  |   8 +-
 .../streaming/CarbonStreamRecordWriter.java |   6 +-
 .../carbondata/hadoop/util/SchemaReader.java|  18 +-
 .../hadoop/test/util/StoreCreator.java  |  16 +-
 .../presto/CarbondataRecordSetProvider.java |   7 +-
 .../presto/impl/CarbonTableCacheModel.java  |  13 +-
 .../presto/impl/CarbonTableReader.java  |  49 ++--
 .../presto/util/CarbonDataStoreCreator.scala|  10 +-
 .../sdv/generated/MergeIndexTestCase.scala  |   8 +-
 .../dataload/TestLoadDataGeneral.scala  |  11 +-
 .../InsertIntoCarbonTableTestCase.scala |   8 +-
 .../createTable/TestCreateTableAsSelect.scala   |   2 +-
 .../datacompaction/DataCompactionLockTest.scala |   6 +-
 .../MajorCompactionIgnoreInMinorTest.scala  |  12 +-
 .../dataload/TestBatchSortDataLoad.scala|   5 +-
 .../dataload/TestDataLoadWithFileName.scala |   5 +-
 .../dataload/TestGlobalSortDataLoad.scala   |   8 +-
 .../dataretention/DataRetentionTestCase.scala   |   6 +-
 .../TestDataLoadingForPartitionTable.scala  |   4 +-
 .../StandardPartitionTableLoadingTestCase.scala |   6 +-
 .../org/apache/carbondata/api/CarbonStore.scala |   4 +-
 .../carbondata/spark/CarbonSparkFactory.scala   |   2 +-
 .../spark/DictionaryDetailHelper.scala  |   9 +-
 .../spark/rdd/AlterTableAddColumnRDD.scala  |  17 +-
 .../spark/rdd/CarbonGlobalDictionaryRDD.scala   |   6 -
 .../carbondata/spark/util/CommonUtil.scala  |  30 +-
 .../carbondata/spark/util/DataLoadingUtil.scala |   8 +-
 .../spark/util/GlobalDictionaryUtil.scala   |  16 +-
 .../command/carbonTableSchemaCommon.scala   |   9 +-
 .../org/apache/spark/util/PartitionUtils.scala  |   6 +-
 .../spark/rdd/AggregateDataMapCompactor.scala   |  11 +-
 .../spark/rdd/CarbonDataRDDFactory.scala|  14 +-
 .../spark/rdd/CarbonTableCompactor.scala|   8 +-
 .../CarbonAlterTableCompactionCommand.scala |  13 +-
 .../management/CarbonLoadDataCommand.scala  |  12 +-
 .../management/CarbonShowLoadsCommand.scala |   2 +-
 .../management/RefreshCarbonTableCommand.scala  |  17 +-
 .../command/mutation/DeleteExecution.scala  |   7 +-
 .../CarbonAlterTableDropPartitionCommand.scala  |  16 +-
 .../CarbonAlterTableSplitPartitionCommand.scala |  20 +-
 .../preaaggregate/PreAggregateUtil.scala|  35 ++-
 .../CarbonAlterTableAddColumnCommand.scala  |   8 +-
 .../CarbonAlterTableDataTypeChangeCommand.scala |   9 +-
 .../CarbonAlterTableDropColumnCommand.scala |   6 +-
 .../schema/CarbonAlterTableRenameCommand.scala  |  28 +-
 

[22/49] carbondata git commit: [CARBONDATA-2025] Unify all path construction through CarbonTablePath static method

2018-02-26 Thread jackylk
[CARBONDATA-2025] Unify all path construction through CarbonTablePath static 
method

Refactory CarbonTablePath:

1.Remove CarbonStorePath and use CarbonTablePath only.
2.Make CarbonTablePath an utility without object creation, it can avoid 
creating object before using it, thus code is cleaner and GC is less.

This closes #1768


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

Branch: refs/heads/carbonstore-rebase4
Commit: 4d453d4b07389b515abff034d3b7ff06ec64abdb
Parents: 975725a
Author: Jacky Li 
Authored: Wed Jan 31 16:14:27 2018 +0800
Committer: Jacky Li 
Committed: Tue Feb 27 00:20:44 2018 +0800

--
 .../DictionaryColumnUniqueIdentifier.java   |  29 +-
 .../dictionary/ManageDictionaryAndBTree.java|  13 +-
 .../core/metadata/AbsoluteTableIdentifier.java  |   4 +-
 .../core/metadata/schema/table/CarbonTable.java |  11 +-
 .../core/mutate/CarbonUpdateUtil.java   |  58 ++--
 .../core/scan/executor/util/QueryUtil.java  |   7 +-
 .../core/service/CarbonCommonFactory.java   |  16 -
 .../carbondata/core/service/PathService.java|  35 ---
 .../core/service/impl/PathFactory.java  |  50 
 .../statusmanager/SegmentStatusManager.java |  30 +-
 .../SegmentUpdateStatusManager.java |  59 ++--
 .../apache/carbondata/core/util/CarbonUtil.java |  60 ++--
 .../util/path/CarbonSharedDictionaryPath.java   |  71 -
 .../core/util/path/CarbonStorePath.java |  71 -
 .../core/util/path/CarbonTablePath.java | 291 ++-
 .../dictionary/AbstractDictionaryCacheTest.java |  11 +-
 .../dictionary/ForwardDictionaryCacheTest.java  |   6 +-
 .../dictionary/ReverseDictionaryCacheTest.java  |   6 +-
 .../reader/CarbonDictionaryReaderImplTest.java  |   8 -
 .../CarbonFormatDirectoryStructureTest.java |  18 +-
 .../path/CarbonFormatSharedDictionaryTest.java  |  44 ---
 .../writer/CarbonDictionaryWriterImplTest.java  |  19 +-
 .../CarbonBatchSparkStreamingExample.scala  |   9 +-
 .../CarbonStructuredStreamingExample.scala  |  11 +-
 .../hadoop/api/CarbonTableInputFormat.java  |   8 +-
 .../streaming/CarbonStreamRecordWriter.java |   6 +-
 .../carbondata/hadoop/util/SchemaReader.java|  18 +-
 .../hadoop/test/util/StoreCreator.java  |  16 +-
 .../presto/CarbondataRecordSetProvider.java |   7 +-
 .../presto/impl/CarbonTableCacheModel.java  |  13 +-
 .../presto/impl/CarbonTableReader.java  |  49 ++--
 .../presto/util/CarbonDataStoreCreator.scala|  10 +-
 .../sdv/generated/MergeIndexTestCase.scala  |   8 +-
 .../dataload/TestLoadDataGeneral.scala  |  11 +-
 .../InsertIntoCarbonTableTestCase.scala |   8 +-
 .../createTable/TestCreateTableAsSelect.scala   |   2 +-
 .../datacompaction/DataCompactionLockTest.scala |   6 +-
 .../MajorCompactionIgnoreInMinorTest.scala  |  12 +-
 .../dataload/TestBatchSortDataLoad.scala|   5 +-
 .../dataload/TestDataLoadWithFileName.scala |   5 +-
 .../dataload/TestGlobalSortDataLoad.scala   |   8 +-
 .../dataretention/DataRetentionTestCase.scala   |   6 +-
 .../TestDataLoadingForPartitionTable.scala  |   4 +-
 .../StandardPartitionTableLoadingTestCase.scala |   6 +-
 .../org/apache/carbondata/api/CarbonStore.scala |   4 +-
 .../carbondata/spark/CarbonSparkFactory.scala   |   2 +-
 .../spark/DictionaryDetailHelper.scala  |   9 +-
 .../spark/rdd/AlterTableAddColumnRDD.scala  |  17 +-
 .../spark/rdd/CarbonGlobalDictionaryRDD.scala   |   6 -
 .../carbondata/spark/util/CommonUtil.scala  |  30 +-
 .../carbondata/spark/util/DataLoadingUtil.scala |   8 +-
 .../spark/util/GlobalDictionaryUtil.scala   |  16 +-
 .../command/carbonTableSchemaCommon.scala   |   9 +-
 .../org/apache/spark/util/PartitionUtils.scala  |   6 +-
 .../spark/rdd/AggregateDataMapCompactor.scala   |  11 +-
 .../spark/rdd/CarbonDataRDDFactory.scala|  14 +-
 .../spark/rdd/CarbonTableCompactor.scala|   8 +-
 .../CarbonAlterTableCompactionCommand.scala |  13 +-
 .../management/CarbonLoadDataCommand.scala  |  12 +-
 .../management/CarbonShowLoadsCommand.scala |   2 +-
 .../management/RefreshCarbonTableCommand.scala  |  17 +-
 .../command/mutation/DeleteExecution.scala  |   7 +-
 .../CarbonAlterTableDropPartitionCommand.scala  |  16 +-
 .../CarbonAlterTableSplitPartitionCommand.scala |  20 +-
 .../preaaggregate/PreAggregateUtil.scala|  35 ++-
 .../CarbonAlterTableAddColumnCommand.scala  |   8 +-
 .../CarbonAlterTableDataTypeChangeCommand.scala |   9 +-
 .../CarbonAlterTableDropColumnCommand.scala |   6 +-
 .../schema/CarbonAlterTableRenameCommand.scala  |  28 +-