You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by je...@apache.org on 2018/11/27 22:05:35 UTC

[incubator-pinot] branch adlFix created (now 87488bf)

This is an automated email from the ASF dual-hosted git repository.

jenniferdai pushed a change to branch adlFix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.


      at 87488bf  Fixing AzurePinotFS move api

This branch includes the following new commits:

     new 87488bf  Fixing AzurePinotFS move api

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[incubator-pinot] 01/01: Fixing AzurePinotFS move api

Posted by je...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

jenniferdai pushed a commit to branch adlFix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git

commit 87488bf52bf65f365a17c9ad446f9f2644854060
Author: Jennifer Dai <jd...@linkedin.com>
AuthorDate: Tue Nov 27 14:05:23 2018 -0800

    Fixing AzurePinotFS move api
---
 .../src/main/java/com/linkedin/pinot/filesystem/AzurePinotFS.java      | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/pinot-azure-filesystem/src/main/java/com/linkedin/pinot/filesystem/AzurePinotFS.java b/pinot-azure-filesystem/src/main/java/com/linkedin/pinot/filesystem/AzurePinotFS.java
index 796d4a0..0a7e84b 100644
--- a/pinot-azure-filesystem/src/main/java/com/linkedin/pinot/filesystem/AzurePinotFS.java
+++ b/pinot-azure-filesystem/src/main/java/com/linkedin/pinot/filesystem/AzurePinotFS.java
@@ -99,8 +99,7 @@ public class AzurePinotFS extends PinotFS {
     if (exists(dstUri) && !overwrite) {
       return false;
     }
-    //rename the file
-    return _adlStoreClient.rename(srcUri.getPath(), dstUri.getPath());
+    return copy(srcUri, dstUri) && delete(srcUri, true);
   }
 
   @Override


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org