683d5423e80fcd429152079568dd9aa578faca4f
Files Diff
- Added: 1
- Removed: 0
- Renamed: 0
- Modified: 17
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Rename Method | 2 | ||
Rename Method public extractRegularColumnHandles(regularColumnMappings List<ColumnMapping>, doCoercion boolean) : List<HiveColumnHandle> renamed to public toColumnHandles(regularColumnMappings List<ColumnMapping>, doCoercion boolean) : List<HiveColumnHandle> in class com.facebook.presto.hive.HivePageSourceProvider.ColumnMapping | From | To | |
Rename Method public extractRegularColumnMappings(columnMappings List<ColumnMapping>) : List<ColumnMapping> renamed to public extractRegularAndInterimColumnMappings(columnMappings List<ColumnMapping>) : List<ColumnMapping> in class com.facebook.presto.hive.HivePageSourceProvider.ColumnMapping | From | To | |
Rename Variable | 2 | ||
Rename Variable regularColumnMappings : List<ColumnMapping> to regularAndInterimColumnMappings : List<ColumnMapping> in method public createHivePageSource(cursorProviders Set<HiveRecordCursorProvider>, pageSourceFactories Set<HivePageSourceFactory>, configuration Configuration, session ConnectorSession, path Path, bucketNumber OptionalInt, start long, length long, fileSize long, schema Properties, effectivePredicate TupleDomain<HiveColumnHandle>, hiveColumns List<HiveColumnHandle>, partitionKeys List<HivePartitionKey>, hiveStorageTimeZone DateTimeZone, typeManager TypeManager, columnCoercions Map<Integer,HiveType>, bucketConversion Optional<BucketConversion>) : Optional<ConnectorPageSource> in class com.facebook.presto.hive.HivePageSourceProvider | From | To | |
Rename Variable blocks : Block[] to adaptedBlocks : Block[] in method public getNextPage() : Page in class com.facebook.presto.hive.HivePageSource | From | To | |
Rename Attribute | 1 | ||
Rename Attribute bucketSplitInfo : Optional<BucketSplitInfo> to tableBucketInfo : Optional<BucketSplitInfo> in class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To | |
Extract Variable | 4 | ||
Extract Variable partitionBucketColumns : List<String> in method private getPartitionMetadata(metastore SemiTransactionalHiveMetastore, table Table, tableName SchemaTableName, hivePartitions List<HivePartition>, bucketProperty Optional<HiveBucketProperty>) : Iterable<HivePartitionMetadata> from class com.facebook.presto.hive.HiveSplitManager | From | To | |
Extract Variable partitionBucketCount : int in method private getPartitionMetadata(metastore SemiTransactionalHiveMetastore, table Table, tableName SchemaTableName, hivePartitions List<HivePartition>, bucketProperty Optional<HiveBucketProperty>) : Iterable<HivePartitionMetadata> from class com.facebook.presto.hive.HiveSplitManager | From | To | |
Extract Variable tableBucketColumns : List<String> in method private getPartitionMetadata(metastore SemiTransactionalHiveMetastore, table Table, tableName SchemaTableName, hivePartitions List<HivePartition>, bucketProperty Optional<HiveBucketProperty>) : Iterable<HivePartitionMetadata> from class com.facebook.presto.hive.HiveSplitManager | From | To | |
Extract Variable tableBucketCount : int in method private getPartitionMetadata(metastore SemiTransactionalHiveMetastore, table Table, tableName SchemaTableName, hivePartitions List<HivePartition>, bucketProperty Optional<HiveBucketProperty>) : Iterable<HivePartitionMetadata> from class com.facebook.presto.hive.HiveSplitManager | From | To | |
Extract Method | 3 | ||
Extract Method private testingPrincipalPrivilege(tableOwner String) : PrincipalPrivileges extracted from protected doTestMismatchSchemaTable(schemaTableName SchemaTableName, storageFormat HiveStorageFormat, tableBefore List<ColumnMetadata>, dataBefore MaterializedResult, tableAfter List<ColumnMetadata>, dataAfter MaterializedResult) : void in class com.facebook.presto.hive.AbstractTestHiveClient | From | To | |
Extract Method private createEmptyTable(schemaTableName SchemaTableName, hiveStorageFormat HiveStorageFormat, columns List<Column>, partitionColumns List<Column>, bucketProperty Optional<HiveBucketProperty>) : void extracted from protected createEmptyTable(schemaTableName SchemaTableName, hiveStorageFormat HiveStorageFormat, columns List<Column>, partitionColumns List<Column>) : void in class com.facebook.presto.hive.AbstractTestHiveClient | From | To | |
Extract Method private testingPrincipalPrivilege(tableOwner String) : PrincipalPrivileges extracted from protected createEmptyTable(schemaTableName SchemaTableName, hiveStorageFormat HiveStorageFormat, columns List<Column>, partitionColumns List<Column>) : void in class com.facebook.presto.hive.AbstractTestHiveClient | From | To | |
Inline Variable | 1 | ||
Inline Variable coercedBlock : Block in method public load(lazyBlock LazyBlock) : void from class com.facebook.presto.hive.HivePageSource.CoercionLazyBlockLoader | From | To | |
Rename Parameter | 1 | ||
Rename Parameter bucketSplitInfo : Optional<BucketSplitInfo> to tableBucketInfo : Optional<BucketSplitInfo> in method public BackgroundHiveSplitLoader(table Table, partitions Iterable<HivePartitionMetadata>, compactEffectivePredicate TupleDomain<? extends ColumnHandle>, tableBucketInfo Optional<BucketSplitInfo>, session ConnectorSession, hdfsEnvironment HdfsEnvironment, namenodeStats NamenodeStats, directoryLister DirectoryLister, executor Executor, loaderConcurrency int, recursiveDirWalkerEnabled boolean) in class com.facebook.presto.hive.BackgroundHiveSplitLoader | From | To |