You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2019/08/18 16:59:24 UTC

[GitHub] [zeppelin] alexott opened a new pull request #3428: [ZEPPELIN-4305] LocalStorageConfig.atomicWriteToFile throws exception

alexott opened a new pull request #3428: [ZEPPELIN-4305] LocalStorageConfig.atomicWriteToFile throws exception
URL: https://github.com/apache/zeppelin/pull/3428
 
 
   ### What is this PR for?
   
   The hotfix that was made several weeks ago changed behavior of LocalStorageConfig.atomicWriteToFile, that started to atomically move files to destination. But this works without errors only when temporary directory and destination are on the same disk. When they are on different disks, atomic move isn't possible, so the exception is thrown.
   
   This PR fixes this by performing non-atomic move to temp file on destination file system, and then atomically rename it to destination file.
   
   ### What type of PR is it?
   
   Bug Fix
   
   ### What is the Jira issue?
   
   ZEPPELIN-4305
   
   ### Questions:
   * Does the licenses files need update?
   
   No
   
   * Is there breaking changes for older versions?
   
   No
   
   * Does this needs documentation?
   
   No
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services