qinghui-xu commented on issue #15313: URL: https://github.com/apache/iceberg/issues/15313#issuecomment-4143450594
Hello @mukund-thakur Thanks for looking into this. Maybe my initial description was not clear enough. But as I explained in my [PR](https://github.com/apache/iceberg/pull/15560#issuecomment-4124454437), I was comparing the `Schema` and trying to drop some stale fields, and in the case of map fields, both map field and its children ("key" + "value") are dropped altogher. I agree that the map key as a field should be immutable, as what the spec says. But in the case of dropping map field, I feel it's legit to be able drop "key" and "value" fields explicitly as well. -- 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]
