You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by xi...@apache.org on 2022/06/01 10:21:15 UTC

[pinot] branch master updated: Fix metadata push mode in IngestionUtils (#8802)

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

xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 9abf15fd6d Fix metadata push mode in IngestionUtils (#8802)
9abf15fd6d is described below

commit 9abf15fd6dee3323f90a1c6594ca68ebafd8c858
Author: Lars-Kristian Svenøy <46...@users.noreply.github.com>
AuthorDate: Wed Jun 1 11:21:06 2022 +0100

    Fix metadata push mode in IngestionUtils (#8802)
    
    * Fix bug in IngestionUtils creating segment uri to tar map
    
    * Use new String[0] instead of static method reference
---
 .../main/java/org/apache/pinot/segment/local/utils/IngestionUtils.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/IngestionUtils.java b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/IngestionUtils.java
index 1a4695f9a0..450d8bea93 100644
--- a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/IngestionUtils.java
+++ b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/IngestionUtils.java
@@ -240,7 +240,7 @@ public final class IngestionUtils {
           }
           PinotFS outputFileFS = getOutputPinotFS(batchConfig, outputSegmentDirURI);
           Map<String, String> segmentUriToTarPathMap = SegmentPushUtils.getSegmentUriToTarPathMap(outputSegmentDirURI,
-              segmentUploadSpec.getPushJobSpec(), new String[] { segmentTarURIs.toString() });
+              segmentUploadSpec.getPushJobSpec(), segmentTarURIStrs.toArray(new String[0]));
           SegmentPushUtils.sendSegmentUriAndMetadata(segmentUploadSpec, outputFileFS, segmentUriToTarPathMap);
         } catch (RetriableOperationException | AttemptsExceededException e) {
           throw new RuntimeException(String


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