Github user HeartSaVioR commented on the issue:

    https://github.com/apache/spark/pull/22952
  
    @gaborgsomogyi @steveloughran 
    `GlobExpander` only looks like handling `{}` pattern. We need to still deal 
with `*` and `?` which can't be expanded like this. 
    
    It would only work if we would be OK with restricting descendants of 
multiple paths (for now we restrict descendants of one path), so while it would 
help fixing the bug of current patch, it might be still too restrictive.
    
    I think changing Hadoop version because of this costs too much. If we 
really would like to go, only viable solution is copying the code. (Actually we 
can also just reimplement it since its requirements are like a kind of 
assignment, though we may end up with similar code.)


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to