dgvj-work opened a new pull request, #1707: URL: https://github.com/apache/iceberg-go/pull/1707
## Summary - Wire `--catalog sql` through `catalog.Load` so the CLI uses the registered SQL catalog instead of failing with `unrecognized catalog type`. - Add `--sql-driver` / `--sql-dialect` flags and matching `sql-driver` / `sql-dialect` config keys. - Recognize `--catalog dynamodb` with a clear `not implemented` error (type exists; catalog package does not yet). - Document the new flags/config and cover mixed-case SQL plus DynamoDB validation in CLI tests. Fixes #1692 ## Testing - `go test ./config/ ./cmd/iceberg/ -count=1` - `gofmt` clean on touched Go files Made with [Cursor](https://cursor.com) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
