Check behavior when dropping a field that's part of the current partition spec

apache / iceberg-python

Apache PyIceberg

https://py.iceberg.apache.org/

Apache License 2.0

472 stars 174 forks source link

Check behavior when dropping a field that's part of the current partition spec #1293

Open Fokko opened 2 weeks ago

Fokko commented 2 weeks ago

Feature Request / Improvement

While reviewing this: https://github.com/apache/iceberg-python/pull/931/files#diff-23e8153e0fd497a9212215bd2067068f3b56fa071770c7ef326db3d3d03cee9bR467

I believe we should disallow dropping a field when it is part of the current partition spec. We should add checks for both the REST and non-REST catalog.