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 2019/06/21 00:34:33 UTC

[incubator-pinot] branch master updated: Refresh schema when reloading all segments (#4349)

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/incubator-pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 17847db  Refresh schema when reloading all segments (#4349)
17847db is described below

commit 17847db705cabd823c268342ed3ba6e1b2be4de1
Author: haibow <ha...@users.noreply.github.com>
AuthorDate: Thu Jun 20 17:34:28 2019 -0700

    Refresh schema when reloading all segments (#4349)
---
 .../apache/pinot/server/starter/helix/HelixInstanceDataManager.java | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixInstanceDataManager.java b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixInstanceDataManager.java
index 62b70fd..aafeebe 100644
--- a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixInstanceDataManager.java
+++ b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixInstanceDataManager.java
@@ -193,11 +193,7 @@ public class HelixInstanceDataManager implements InstanceDataManager {
     TableConfig tableConfig = ZKMetadataProvider.getTableConfig(_propertyStore, tableNameWithType);
     Preconditions.checkNotNull(tableConfig);
 
-    Schema schema = null;
-    // For OFFLINE table, try to get schema for default columns
-    if (TableNameBuilder.OFFLINE.tableHasTypeSuffix(tableNameWithType)) {
-      schema = ZKMetadataProvider.getTableSchema(_propertyStore, tableNameWithType);
-    }
+    Schema schema = ZKMetadataProvider.getTableSchema(_propertyStore, tableNameWithType);
 
     for (SegmentMetadata segmentMetadata : getAllSegmentsMetadata(tableNameWithType)) {
       reloadSegment(tableNameWithType, segmentMetadata, tableConfig, schema);


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