You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by sn...@apache.org on 2018/11/14 19:35:12 UTC
[incubator-pinot] 01/01: Fix typo in BaseRoutingTableBuilder
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-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org