Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/22328#discussion_r215000161
--- Diff: data/mllib/images/images/license.txt ---
@@ -0,0 +1,13 @@
+The images in the folder "kittens" are under the creative commons CC0
license, or no rights reserved:
--- End diff --
update to comment above:
I don't think you need duplicate images in the PR - in the old tests you
can just specify the path up to the cls/date folder. There might be a few minor
changes to the tests but I think that would be a better strategy than to have
duplicate images in source.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
