Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18805#discussion_r139635152
  
    --- Diff: LICENSE ---
    @@ -269,6 +269,7 @@ The text of each license is also included at 
licenses/LICENSE-[project].txt.
          (BSD 3 Clause) d3.min.js 
(https://github.com/mbostock/d3/blob/master/LICENSE)
          (BSD 3 Clause) DPark 
(https://github.com/douban/dpark/blob/master/LICENSE)
          (BSD 3 Clause) CloudPickle 
(https://github.com/cloudpipe/cloudpickle/blob/master/LICENSE)
    +     (BSD 2 Clause) Zstd-jni 
(https://github.com/luben/zstd-jni/blob/master/LICENSE)
    --- End diff --
    
    ZStandard itself is also included, not just zstd-jni. We'd need an entry 
for it, and its LICENSE as below.
    https://github.com/luben/zstd-jni/tree/master/src/main/native


---

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

Reply via email to