kevinjqliu commented on PR #424: URL: https://github.com/apache/iceberg-go/pull/424#issuecomment-2869302773
Thanks for the contribution @lliangyu-lin! I just tested this locally against pyiceberg's integration test's IRC. ``` go run ./cmd/iceberg --uri http://0.0.0.0:8181 create table default.test --schema '[{"name":"id","type":"int","required":false},{"name":"name","type":"string","required":true}]' ``` did not output any messages, even though it ran successfully Another run of the same command outputted: ``` 2025/05/10 18:08:42 failed to create table: AlreadyExistsException: Table already exists: default.test exit status 1 ``` I also noticed that when running `go run ./cmd/iceberg -h`, the `create` command doesn't show up in the `Commands:` section. I think we can address these in a followup PR :) -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
