anthonychengit opened a new pull request, #6315: URL: https://github.com/apache/texera/pull/6315
### What changes were proposed in this PR? This PR removes the unused `hadoop` Iceberg catalog type. Changes: - Remove the `hadoop` catalog branch from `IcebergCatalogInstance`. - Remove `IcebergUtil.createHadoopCatalog`. - Remove the unused `HadoopCatalog` import while keeping `HadoopFileIO`. - Update `IcebergTableWriterSpec` to use the supported postgres catalog helper. - Update the `storage.conf` catalog-type comment from `hadoop, rest, or postgres` to `rest or postgres`. ### Any related issues, documentation, discussions? Closes #6150 ### How was this PR tested? Tested with: - `sbt "WorkflowCore / scalafmtOnly"` - `sbt "WorkflowCore / compile"` Also attempted: - `sbt "WorkflowCore / testOnly org.apache.texera.amber.core.storage.result.iceberg.IcebergTableWriterSpec"` ### Was this PR authored or co-authored using generative AI tooling? Generated-by: ChatGPT was used for planning and review assistance. I reviewed and tested the final code before submission. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
