prodeezy commented on issue #123: Add support for struct field based filtering
URL: https://github.com/apache/incubator-iceberg/pull/123#issuecomment-471499273
 
 
   > General API design related question - how do we bubble up in the API layer 
the fact that we are supporting the struct field based filtering but not other 
nested field types filtering? ..  I am just curious what a client's 
expectations should be that the API exposes this new capacity in a coherent 
manner.
   
   @fbocse  Currently it would be implied. The only feedback the client gets 
right now is in the Physical plan, that this filter is pushed down to Iceberg 
Scan level and if the client inspects the Scan (using  
iceTable.newScan().filter(structFilterExp).planFiles ) , should be able to 
notice the appropriate files being skipped. AFAIK, this is consistent with how 
top-level field based filter expressions currently communicate to the client.
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to