[incubator-pinot] 01/01: Fix typo in BaseRoutingTableBuilder

[snlee]

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

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

commit d540048d03792e100855eb324170b40dd59ff017
Author: Seunghyun Lee <sn...@linkedin.com>
AuthorDate: Wed Nov 14 11:33:43 2018 -0800

    Fix typo in BaseRoutingTableBuilder
---
 .../pinot/broker/routing/builder/BaseRoutingTableBuilder.java | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git 
a/pinot-broker/src/main/java/com/linkedin/pinot/broker/routing/builder/BaseRoutingTableBuilder.java
 
b/pinot-broker/src/main/java/com/linkedin/pinot/broker/routing/builder/BaseRoutingTableBuilder.java
index 292b03f..c00d30d 100644
--- 
a/pinot-broker/src/main/java/com/linkedin/pinot/broker/routing/builder/BaseRoutingTableBuilder.java
+++ 
b/pinot-broker/src/main/java/com/linkedin/pinot/broker/routing/builder/BaseRoutingTableBuilder.java
@@ -24,7 +24,6 @@ import com.linkedin.pinot.common.utils.CommonConstants;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
-import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import java.util.Map;
@@ -47,7 +46,7 @@ public abstract class BaseRoutingTableBuilder implements 
RoutingTableBuilder {
   protected final Random _random = new Random();
   private BrokerMetrics _brokerMetrics;
   private String _tableName;
-  private boolean _enableDyanmicComputing;
+  private boolean _enableDynamicComputing;
 
   // Set variable as volatile so all threads can get the up-to-date routing 
tables
   // Routing tables are used for storing pre-computed routing table
@@ -66,8 +65,8 @@ public abstract class BaseRoutingTableBuilder implements 
RoutingTableBuilder {
     RoutingConfig routingConfig = tableConfig.getRoutingConfig();
     if (routingConfig != null) {
       Map<String, String> routingOption = 
routingConfig.getRoutingTableBuilderOptions();
-      _enableDyanmicComputing = 
Boolean.valueOf(routingOption.get(RoutingConfig.ENABLE_DYNAMIC_COMPUTING_KEY));
-      if (_enableDyanmicComputing) {
+      _enableDynamicComputing = 
Boolean.valueOf(routingOption.get(RoutingConfig.ENABLE_DYNAMIC_COMPUTING_KEY));
+      if (_enableDynamicComputing) {
         LOGGER.info("Dynamic routing table computation is enabled for table 
{}", _tableName);
       }
     }
@@ -114,7 +113,7 @@ public abstract class BaseRoutingTableBuilder implements 
RoutingTableBuilder {
     Map<String, List<String>> segmentToServersMap =
         computeSegmentToServersMapFromExternalView(externalView, 
instanceConfigs);
 
-    if (_enableDyanmicComputing) {
+    if (_enableDynamicComputing) {
       // When dynamic computing is enabled, cache the mapping
       _segmentToServersMap = segmentToServersMap;
     } else {
@@ -126,7 +125,7 @@ public abstract class BaseRoutingTableBuilder implements 
RoutingTableBuilder {
 
   @Override
   public Map<String, List<String>> getRoutingTable(RoutingTableLookupRequest 
request) {
-    if (_enableDyanmicComputing) {
+    if (_enableDynamicComputing) {
       // Copy the pointer for snapshot since the pointer for segment to 
servers map can change at anytime
       Map<String, List<String>> segmentToServersMap = _segmentToServersMap;
 


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