You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2022/11/30 17:29:18 UTC

[pinot] branch master updated: Bug fix: Get correct primary key count (#9876)

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 28f337a6af Bug fix: Get correct primary key count (#9876)
28f337a6af is described below

commit 28f337a6af255d06a4b031a5181fdc2174c40408
Author: Kartik Khare <kh...@gmail.com>
AuthorDate: Wed Nov 30 22:59:12 2022 +0530

    Bug fix: Get correct primary key count (#9876)
---
 .../local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java
index 7b8cd6039d..e11c1899f3 100644
--- a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java
+++ b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java
@@ -65,7 +65,7 @@ public class ConcurrentMapPartitionUpsertMetadataManager extends BasePartitionUp
 
   @Override
   protected long getNumPrimaryKeys() {
-    return _primaryKeyColumns.size();
+    return _primaryKeyToRecordLocationMap.size();
   }
 
   @Override


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