Github user mhamilton723 commented on the issue:

    https://github.com/apache/spark/pull/22328
  
    @WeichenXu123. Awesome work! I have not had a chance to go through this in 
depth but I did this in the originating project, [MMLSpark](www.aka.ms/spark), 
a while back and have been meaning to ship it up to Spark Core:
    
    https://github.com/Azure/mmlspark/tree/master/src/io/image
    
    We built ours on the same functionality for Binary Files and it might be 
wise to do the same here if you have not already. The Binary File API is a bit 
more general as it supports reading many different kinds of files. 
    
    https://github.com/Azure/mmlspark/tree/master/src/io/binary


---

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

Reply via email to