[GitHub] [spark] yaooqinn commented on pull request #35117: [SPARK-37826][SQL] Use zstd codec name in ORC file names for hive orc impl

[GitBox <gi...@apache.org>]

yaooqinn commented on pull request #35117:
URL: https://github.com/apache/spark/pull/35117#issuecomment-1007119289


   > spark.range(3).write.option("compression", "zstd").orc("/tmp/after")
   
   Yes, I have tested this PR with the spark-sql shell while sending this PR, and the result is correct.
   
   I modified and verified the case you provided above w/ and w/o this PR, the warning always shows and not impact on the correctness. So I guess it's not related to this PR.
   
   Currently, my guess is SingleDirectoryDataWriter call `newOutputWriter()` during `init` and `write` twice, and the later seems to try to visit a non-existent file in the former. I will take a look later.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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