Github user foxish commented on the issue:

    https://github.com/apache/spark/pull/21095
  
    @madanadit, sorry about the long turnaround time here. I made a cursory 
pass over it - and it looks good. Before getting deeper into the code, I think 
we need more exhaustive unit testing for non-hostpath type volumes, and a 
couple of [e2e 
tests](https://github.com/apache-spark-on-k8s/spark-integration). Once those 
are done, I'm also happy to do a more detailed review here.



---

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

Reply via email to