gengliangwang commented on pull request #28841:
URL: https://github.com/apache/spark/pull/28841#issuecomment-647880453


   > The need arises since we only have one filtering strategy where 
pathGlobFilter is used in PartitioningAwareFileIndex and handling cleanly if we 
now have more than one approach to path filtering so the implementation I'm 
suggesting would be nearly identical to above except moving it to where 
pathGlobFilter is implemented.
   
   @cchighman the option should be effective for both `MetadataLogFileIndex` 
and `InMemoryFileIndex`. That's why I suggest implementing it in 
`PartitioningAwareFileIndex`. I think we can create a method to determine which 
files are matched according to the parameters.


----------------------------------------------------------------
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]



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

Reply via email to