Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22179#discussion_r214491757
  
    --- Diff: pom.xml ---
    @@ -1770,6 +1770,10 @@
                 <groupId>org.apache.hive</groupId>
                 <artifactId>hive-storage-api</artifactId>
               </exclusion>
    +          <exclusion>
    +            <groupId> com.esotericsoftware</groupId>
    +            <artifactId>kryo-shaded</artifactId>
    --- End diff --
    
    Yes. I checked that, @srowen . `org.apache.orc` only uses Kryo constructor, 
`writeObject`, and `readObject` from `kryo-shaded` library. There is no change 
for them.
    
    **WRITE**
    ```
    (new Kryo()).writeObject(out, sarg);
    ```
    **READ**
    ```
    ... = (new Kryo()).readObject(new Input(sargBytes), 
SearchArgumentImpl.class);
    ```


---

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

Reply via email to