[pinot] branch master updated: Bug Fix: Handle empty validDocId snapshots during reloading (#10406)

[ja...@apache.org]

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

jackie 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 be9dfe74ad Bug Fix: Handle empty validDocId snapshots during reloading (#10406)
be9dfe74ad is described below

commit be9dfe74adbfe7607f0ece69f8bc22b0d7c4a913
Author: Kartik Khare <kh...@gmail.com>
AuthorDate: Sun Mar 12 06:15:39 2023 +0530

    Bug Fix: Handle empty validDocId snapshots during reloading (#10406)
---
 .../pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java   | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java
index 066ef9cac1..e979823c2e 100644
--- a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java
+++ b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java
@@ -124,6 +124,7 @@ public abstract class BasePartitionUpsertMetadataManager implements PartitionUps
       if (validDocIds != null && validDocIds.isEmpty()) {
         _logger.info("Skip adding segment: {} without valid doc, current primary key count: {}",
             segment.getSegmentName(), getNumPrimaryKeys());
+       immutableSegmentImpl.enableUpsert(this, new ThreadSafeMutableRoaringBitmap());
         return;
       }
     } else {


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