huan233usc commented on PR #2575:
URL: https://github.com/apache/iceberg-rust/pull/2575#issuecomment-4693895114
> Hi @huan233usc , thanks for the contribution. Throwing errors on partition
transforms looks good to me. However, I'm not sure if this is a problem that we
want to solve at this point.
>
> Currently we don't support `CREATE EXTERNAL TABLE`/`register_table`, and I
think we should tackle #2021 first before coming to this. wdyt?
Hi @CTTY
Makes sense, thanks for the context.
Based on my observation, from a user perspective, the ideal priority would
probably be:
1. `CREATE TABLE`
-- this is isn't really doable with stock DataFusion today unless we
make
2. `CREATE EXTERNAL TABLE ... LOCATION ...` where `LOCATION` points to a
storage path (creating a new table there)
-- iiuc is https://github.com/apache/iceberg-rust/issues/2021 is mainly
about.
3. `CREATE EXTERNAL TABLE ... LOCATION ...` where `LOCATION` points to an
existing metadata JSON / snapshot (read-only registration) (this PR handle,
stepping back a bit this pr seems a bit redundant/unnecessary )
Let me know if anything I could help with
https://github.com/apache/iceberg-rust/issues/2021?
Thanks
--
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]
