hililiwei opened a new pull request, #4824:
URL: https://github.com/apache/iceberg/pull/4824

   ## What is the purpose of the change
   
   Currently, iceberg flink unit tests cannot be started on Windows OS, this 
makes it very inconvenient for developers who use Windows as their workhorse 
(like me, huh). This PR aims to make unit tests compatible with Windows and 
Linux at the same time.
   
   
   ## Brief change log
   
   The main reason for incompatibility with Windows is the error related to the 
file path, specifically due to: "URISyntaxException: Relative Path in Absolute 
URI". For example, `file:C:/Users/` should be changed to `file:/C:/Users/`.
   
   I've added a method to handle windows that will add "/" to paths.
   
   ## Verifying this change
   
   On my own Windows PC, verifying this change with the following command:
   ```
   ./gradlew -DsparkVersions= -DhiveVersions= -DflinkVersions=1.15 
:iceberg-flink:iceberg-flink-1.15:check 
:iceberg-flink:iceberg-flink-runtime-1.15:check -Pquick=true -x javadoc
   ```


-- 
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]

Reply via email to