Shoothzj commented on pull request #12329:
URL: https://github.com/apache/pulsar/pull/12329#issuecomment-940762541


   > Could you explain why it cannot run on Windows? I see the core change is 
changing
   > 
   > ```java
   > "file://" + tokenFile
   > ```
   > 
   > to
   > 
   > ```java
   > "file:///" + tokenFile.toString().replace('\\', '/')
   > ```
   > 
   > Could you give an example for what `File.createTempFile` could return on 
Windows?
   
   
   
   > Could you explain why it cannot run on Windows? I see the core change is 
changing
   > 
   > ```java
   > "file://" + tokenFile
   > ```
   > 
   > to
   > 
   > ```java
   > "file:///" + tokenFile.toString().replace('\\', '/')
   > ```
   > 
   > Could you give an example for what `File.createTempFile` could return on 
Windows?
   
   Run in windows can't recognize format like `file://C\xxx\yyy\zzz`
   
   It should be file:///C and the `\` should be `/`


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


Reply via email to