126fac0194d43028e9e12d079507ab801519a022
Files Diff
- Added: 1
- Removed: 0
- Renamed: 0
- Modified: 8
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Move And Rename Method | 1 | ||
| Move And Rename Method private deleteIfExists(user String, path Path) : boolean from class com.facebook.presto.hive.HiveMetadata to private deleteRecursivelyIfExists(user String, hdfsEnvironment HdfsEnvironment, path Path) : boolean from class com.facebook.presto.hive.metastore.SemiTransactionalHiveMetastore | From | To | |
| Change Parameter Type | 4 | ||
| Change Parameter Type metastore : ExtendedHiveMetastore to metastore : SemiTransactionalHiveMetastore in method public getPartitions(session ConnectorSession, metastore SemiTransactionalHiveMetastore, tableHandle ConnectorTableHandle, effectivePredicate TupleDomain<ColumnHandle>) : HivePartitionResult in class com.facebook.presto.hive.HivePartitionManager | From | To | |
| Change Parameter Type metastore : ExtendedHiveMetastore to metastore : SemiTransactionalHiveMetastore in method private getFilteredPartitionNames(metastore SemiTransactionalHiveMetastore, tableName SchemaTableName, partitionKeys List<HiveColumnHandle>, effectivePredicate TupleDomain<ColumnHandle>) : List<String> in class com.facebook.presto.hive.HivePartitionManager | From | To | |
| Change Parameter Type metastore : ExtendedHiveMetastore to metastore : SemiTransactionalHiveMetastore in method private getTable(metastore SemiTransactionalHiveMetastore, tableName SchemaTableName) : Table in class com.facebook.presto.hive.HivePartitionManager | From | To | |
| Change Parameter Type metastore : ExtendedHiveMetastore to metastore : SemiTransactionalHiveMetastore in method public HiveMetadata(connectorId String, metastore SemiTransactionalHiveMetastore, hdfsEnvironment HdfsEnvironment, partitionManager HivePartitionManager, timeZone DateTimeZone, allowCorruptWritesForTesting boolean, respectTableFormat boolean, bucketExecutionEnabled boolean, bucketWritingEnabled boolean, forceIntegralToBigint boolean, defaultStorageFormat HiveStorageFormat, typeManager TypeManager, locationService LocationService, tableParameterCodec TableParameterCodec, partitionUpdateCodec JsonCodec<PartitionUpdate>, renameExecutor Executor, typeTranslator TypeTranslator, serverInfo ServerInfo) in class com.facebook.presto.hive.HiveMetadata | From | To | |
| Move Method | 3 | ||
| Move Method private deleteIfExists(fileSystem FileSystem, path Path) : boolean from class com.facebook.presto.hive.HiveMetadata to private deleteIfExists(fileSystem FileSystem, path Path) : boolean from class com.facebook.presto.hive.metastore.SemiTransactionalHiveMetastore | From | To | |
| Move Method private isSameOrParent(parent Path, child Path) : boolean from class com.facebook.presto.hive.HiveMetadata to private isSameOrParent(parent Path, child Path) : boolean from class com.facebook.presto.hive.metastore.SemiTransactionalHiveMetastore | From | To | |
| Move Method public renameDirectory(user String, hdfsEnvironment HdfsEnvironment, schemaName String, tableName String, source Path, target Path) : void from class com.facebook.presto.hive.HiveWriteUtils to private renameDirectory(user String, hdfsEnvironment HdfsEnvironment, source Path, target Path, runWhenPathDoesntExist Runnable) : void from class com.facebook.presto.hive.metastore.SemiTransactionalHiveMetastore | From | To | |
| Change Return Type | 1 | ||
| Change Return Type ExtendedHiveMetastore to SemiTransactionalHiveMetastore in method public getMetastore() : SemiTransactionalHiveMetastore in class com.facebook.presto.hive.HiveMetadata | From | To | |
| Move Attribute | 2 | ||
| Move Attribute private log : Logger from class com.facebook.presto.hive.HiveMetadata to private log : Logger from class com.facebook.presto.hive.metastore.SemiTransactionalHiveMetastore | From | To | |
| Move Attribute private PARTITION_COMMIT_BATCH_SIZE : int from class com.facebook.presto.hive.HiveMetadata to private PARTITION_COMMIT_BATCH_SIZE : int from class com.facebook.presto.hive.metastore.SemiTransactionalHiveMetastore | From | To | |
| Extract Class | 1 | ||
| Extract Class com.facebook.presto.hive.metastore.SemiTransactionalHiveMetastore from class com.facebook.presto.hive.HiveMetadata | From | To | |
| Extract Variable | 3 | ||
| Extract Variable locationHandle : LocationHandle in method public beginInsert(session ConnectorSession, tableHandle ConnectorTableHandle) : HiveInsertTableHandle from class com.facebook.presto.hive.HiveMetadata | From | To | |
| Extract Variable locationHandle : LocationHandle in method public beginCreateTable(session ConnectorSession, tableMetadata ConnectorTableMetadata, layout Optional<ConnectorNewTableLayout>) : HiveOutputTableHandle from class com.facebook.presto.hive.HiveMetadata | From | To | |
| Extract Variable metadata : HiveMetadata in method public commit(transaction ConnectorTransactionHandle) : void from class com.facebook.presto.hive.HiveConnector | From | To | |
| Change Attribute Type | 1 | ||
| Change Attribute Type metastore : ExtendedHiveMetastore to metastore : SemiTransactionalHiveMetastore in class com.facebook.presto.hive.HiveMetadata | From | To |