[GitHub] [iceberg] zhongyujiang commented on issue #7065: Cannot delete column with nested field filter

["zhongyujiang (via GitHub)" <gi...@apache.org>]

zhongyujiang commented on issue #7065:
URL: https://github.com/apache/iceberg/issues/7065#issuecomment-1466262033

   I debugged this test locally, and found that it didn't run `canDeleteWhere`, I think this is because the expression in the test cannot be pushed down, after I changed the expression to `complex.c1 = 2`, it threw the same expection. I think exception is unexpected, since this test case already shows that deleting via filter on nested fields is possible (without running `canDeleteUsingMetadata`). But this exception is thrown because `StrictMetricsEvaluator ` cannot handle nested fields. 
   I think we can make `StrictMetricsEvaluator` handle nested fileds properly by removing the nested fields check. Removing it should have no side effects in my opinion, @rdblue can you please confirm this?


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org