f8648a1dcca1f9d03eef4933ef0b1be2c9cce043
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 13
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Change Parameter Type | 4 | ||
| Change Parameter Type bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in method public HivePartition(tableName SchemaTableName, effectivePredicate TupleDomain<HiveColumnHandle>, buckets List<HiveBucket>) in class com.facebook.presto.hive.HivePartition | From | To | |
| Change Parameter Type tableLayoutHandle : ConnectorTableLayoutHandle to tableLayoutHandle : HiveTableLayoutHandle in method protected getPartitions(tableLayoutHandle HiveTableLayoutHandle) : List<?> in class com.facebook.presto.hive.TestHiveIntegrationSmokeTest | From | To | |
| Change Parameter Type bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in method public BackgroundHiveSplitLoader(connectorId String, table Table, partitions Iterable<HivePartitionMetadata>, bucketHandle Optional<HiveBucketHandle>, buckets List<HiveBucket>, session ConnectorSession, hdfsEnvironment HdfsEnvironment, namenodeStats NamenodeStats, directoryLister DirectoryLister, executor Executor, maxPartitionBatchSize int, maxInitialSplits int, recursiveDirWalkerEnabled boolean) in class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
| Change Parameter Type bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in method public HivePartition(tableName SchemaTableName, effectivePredicate TupleDomain<HiveColumnHandle>, partitionId String, keys Map<ColumnHandle,NullableValue>, buckets List<HiveBucket>) in class com.facebook.presto.hive.HivePartition | From | To | |
| Rename Variable | 2 | ||
| Rename Variable bucket : Optional<HiveBucketing.HiveBucket> to buckets : List<HiveBucketing.HiveBucket> in method public getSplits(transaction ConnectorTransactionHandle, session ConnectorSession, layoutHandle ConnectorTableLayoutHandle) : ConnectorSplitSource in class com.facebook.presto.hive.HiveSplitManager | From | To | |
| Rename Variable bucket : Optional<HiveBucketing.HiveBucket> to buckets : List<HiveBucket> in method public getPartitions(session ConnectorSession, metastore SemiTransactionalHiveMetastore, tableHandle ConnectorTableHandle, effectivePredicate TupleDomain<ColumnHandle>) : HivePartitionResult in class com.facebook.presto.hive.HivePartitionManager | From | To | |
| Merge Variable | 1 | ||
| Merge Variable [partitionName : String, partitionKeys : List<HivePartitionKey>] to iterator : HiveFileIterator in method private loadPartition(partition HivePartitionMetadata) : void in class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
| Rename Attribute | 2 | ||
| Rename Attribute bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in class com.facebook.presto.hive.HivePartition | From | To | |
| Rename Attribute bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
| Change Variable Type | 2 | ||
| Change Variable Type bucket : Optional<HiveBucketing.HiveBucket> to buckets : List<HiveBucketing.HiveBucket> in method public getSplits(transaction ConnectorTransactionHandle, session ConnectorSession, layoutHandle ConnectorTableLayoutHandle) : ConnectorSplitSource in class com.facebook.presto.hive.HiveSplitManager | From | To | |
| Change Variable Type bucket : Optional<HiveBucketing.HiveBucket> to buckets : List<HiveBucket> in method public getPartitions(session ConnectorSession, metastore SemiTransactionalHiveMetastore, tableHandle ConnectorTableHandle, effectivePredicate TupleDomain<ColumnHandle>) : HivePartitionResult in class com.facebook.presto.hive.HivePartitionManager | From | To | |
| Extract Variable | 2 | ||
| Extract Variable bucketCount : int in method private loadPartition(partition HivePartitionMetadata) : void from class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
| Extract Variable bucketNumber : int in method private loadPartition(partition HivePartitionMetadata) : void from class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
| Extract Method | 1 | ||
| Extract Method private getHiveTableProperty(tableName String, propertyGetter Function<HiveTableLayoutHandle,Object>) : Object extracted from private getPartitions(tableName String) : List<?> in class com.facebook.presto.hive.TestHiveIntegrationSmokeTest | From | To | |
| Change Attribute Type | 2 | ||
| Change Attribute Type bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in class com.facebook.presto.hive.HivePartition | From | To | |
| Change Attribute Type bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
| Rename Parameter | 3 | ||
| Rename Parameter bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in method public BackgroundHiveSplitLoader(connectorId String, table Table, partitions Iterable<HivePartitionMetadata>, bucketHandle Optional<HiveBucketHandle>, buckets List<HiveBucket>, session ConnectorSession, hdfsEnvironment HdfsEnvironment, namenodeStats NamenodeStats, directoryLister DirectoryLister, executor Executor, maxPartitionBatchSize int, maxInitialSplits int, recursiveDirWalkerEnabled boolean) in class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
| Rename Parameter bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in method public HivePartition(tableName SchemaTableName, effectivePredicate TupleDomain<HiveColumnHandle>, partitionId String, keys Map<ColumnHandle,NullableValue>, buckets List<HiveBucket>) in class com.facebook.presto.hive.HivePartition | From | To | |
| Rename Parameter bucket : Optional<HiveBucket> to buckets : List<HiveBucket> in method public HivePartition(tableName SchemaTableName, effectivePredicate TupleDomain<HiveColumnHandle>, buckets List<HiveBucket>) in class com.facebook.presto.hive.HivePartition | From | To |