skytin1004 commented on code in PR #7954:
URL: https://github.com/apache/iceberg/pull/7954#discussion_r1250478761
##########
orc/src/test/java/org/apache/iceberg/orc/TestOrcDataWriter.java:
##########
@@ -141,7 +137,7 @@ public void testUsingFileIO() throws IOException {
// local files that would be the LocalFileSystem. To prevent this we use
the Proxy classes to
// use a scheme `dummy` that is not handled. Note that Hadoop 2.7.3 throws
IOException
// while latest Hadoop versions throw UnsupportedFileSystemException
(extends IOException)
- ProxyOutputFile outFile = new
ProxyOutputFile(Files.localOutput(temp.newFile()));
+ ProxyOutputFile outFile = new
ProxyOutputFile(Files.localOutput(String.valueOf(temp.isFile())));
Review Comment:
I changed the variable to go inside to fit the context.
`String.valueOf(temp.isFile())` to `temp`
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
