Merge pull request #10 from ytalashko/master

Remove redundant conversions into streams
This commit is contained in:
Andrew Shvayka 2016-12-11 09:55:54 +02:00 committed by GitHub
commit a4ae4162a3
26 changed files with 67 additions and 75 deletions

View File

@ -121,7 +121,7 @@ public class AppActor extends ContextAwareActor {
private void broadcast(Object msg) {
pluginManager.broadcast(msg);
tenantActors.values().stream().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
tenantActors.values().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
}
private void onToRuleMsg(ToRuleActorMsg msg) {

View File

@ -16,7 +16,6 @@
package org.thingsboard.server.actors.rule;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.Set;
@ -26,7 +25,7 @@ public class SimpleRuleActorChain implements RuleActorChain {
public SimpleRuleActorChain(Set<RuleActorMetaData> ruleSet) {
rules = new ArrayList<>(ruleSet);
Collections.sort(rules, RuleActorMetaData.RULE_ACTOR_MD_COMPARATOR);
rules.sort(RuleActorMetaData.RULE_ACTOR_MD_COMPARATOR);
}
public int size() {

View File

@ -111,7 +111,7 @@ class ASyncMsgProcessor extends AbstractSessionActorMsgProcessor {
Optional<ServerAddress> newTargetServer = systemContext.getRoutingService().resolve(getDeviceId());
if (!newTargetServer.equals(currentTargetServer)) {
currentTargetServer = newTargetServer;
pendingMap.values().stream().forEach(v -> {
pendingMap.values().forEach(v -> {
forwardToAppActor(context, v, currentTargetServer);
if (currentTargetServer.isPresent()) {
logger.debug("[{}] Forwarded msg to new server: {}", sessionId, currentTargetServer.get());

View File

@ -66,7 +66,7 @@ public class SessionManagerActor extends ContextAwareActor {
}
private void broadcast(Object msg) {
sessionActors.values().stream().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
sessionActors.values().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
}
private void onSessionTimeout(SessionTimeoutMsg msg) {

View File

@ -19,7 +19,6 @@ import akka.actor.ActorContext;
import akka.actor.ActorRef;
import akka.actor.Scheduler;
import akka.event.LoggingAdapter;
import com.fasterxml.jackson.core.JsonProcessingException;
import com.fasterxml.jackson.databind.JsonNode;
import com.fasterxml.jackson.databind.ObjectMapper;
import lombok.AllArgsConstructor;

View File

@ -15,9 +15,9 @@
*/
package org.thingsboard.server.actors.shared.plugin;
import java.util.HashMap;
import java.util.Map;
import akka.actor.ActorContext;
import akka.actor.ActorRef;
import akka.actor.Props;
import lombok.extern.slf4j.Slf4j;
import org.thingsboard.server.actors.ActorSystemContext;
import org.thingsboard.server.actors.plugin.PluginActor;
@ -29,12 +29,9 @@ import org.thingsboard.server.common.data.page.PageDataIterable;
import org.thingsboard.server.common.data.page.PageDataIterable.FetchFunction;
import org.thingsboard.server.common.data.plugin.PluginMetaData;
import org.thingsboard.server.dao.plugin.PluginService;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import akka.actor.ActorContext;
import akka.actor.ActorRef;
import akka.actor.Props;
import java.util.HashMap;
import java.util.Map;
@Slf4j
public abstract class PluginManager {
@ -64,17 +61,13 @@ public abstract class PluginManager {
abstract TenantId getTenantId();
public ActorRef getOrCreatePluginActor(ActorContext context, PluginId pluginId) {
ActorRef pluginActor = pluginActors.get(pluginId);
if (pluginActor == null) {
pluginActor = context.actorOf(Props.create(new PluginActor.ActorCreator(systemContext, getTenantId(), pluginId))
.withDispatcher(DefaultActorService.PLUGIN_DISPATCHER_NAME), pluginId.toString());
pluginActors.put(pluginId, pluginActor);
}
return pluginActor;
return pluginActors.computeIfAbsent(pluginId, pId ->
context.actorOf(Props.create(new PluginActor.ActorCreator(systemContext, getTenantId(), pId))
.withDispatcher(DefaultActorService.PLUGIN_DISPATCHER_NAME), pId.toString()));
}
public void broadcast(Object msg) {
pluginActors.values().stream().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
pluginActors.values().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
}
public void remove(PluginId id) {

View File

@ -20,7 +20,6 @@ import org.thingsboard.server.common.data.id.TenantId;
import org.thingsboard.server.common.data.page.PageDataIterable.FetchFunction;
import org.thingsboard.server.common.data.plugin.PluginMetaData;
import org.thingsboard.server.dao.plugin.BasePluginService;
import org.thingsboard.server.dao.plugin.PluginService;
public class SystemPluginManager extends PluginManager {
@ -30,7 +29,7 @@ public class SystemPluginManager extends PluginManager {
@Override
FetchFunction<PluginMetaData> getFetchPluginsFunction() {
return link -> pluginService.findSystemPlugins(link);
return pluginService::findSystemPlugins;
}
@Override

View File

@ -18,8 +18,7 @@ package org.thingsboard.server.actors.shared.rule;
import akka.actor.ActorContext;
import akka.actor.ActorRef;
import akka.actor.Props;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import lombok.extern.slf4j.Slf4j;
import org.thingsboard.server.actors.ActorSystemContext;
import org.thingsboard.server.actors.rule.RuleActor;
import org.thingsboard.server.actors.rule.RuleActorChain;
@ -38,10 +37,9 @@ import org.thingsboard.server.dao.rule.RuleService;
import java.util.*;
@Slf4j
public abstract class RuleManager {
protected static final Logger logger = LoggerFactory.getLogger(RuleManager.class);
protected final ActorSystemContext systemContext;
protected final RuleService ruleService;
protected final Map<RuleId, ActorRef> ruleActors;
@ -63,11 +61,11 @@ public abstract class RuleManager {
ruleMap = new HashMap<>();
for (RuleMetaData rule : ruleIterator) {
logger.debug("[{}] Creating rule actor {}", rule.getId(), rule);
log.debug("[{}] Creating rule actor {}", rule.getId(), rule);
ActorRef ref = getOrCreateRuleActor(context, rule.getId());
RuleActorMetaData actorMd = RuleActorMetaData.systemRule(rule.getId(), rule.getWeight(), ref);
ruleMap.put(rule, actorMd);
logger.debug("[{}] Rule actor created.", rule.getId());
log.debug("[{}] Rule actor created.", rule.getId());
}
refreshRuleChain();
@ -79,8 +77,11 @@ public abstract class RuleManager {
rule = systemContext.getRuleService().findRuleById(ruleId);
}
if (rule == null) {
rule = ruleMap.keySet().stream().filter(r -> r.getId().equals(ruleId)).findFirst().orElse(null);
rule.setState(ComponentLifecycleState.SUSPENDED);
rule = ruleMap.keySet().stream()
.filter(r -> r.getId().equals(ruleId))
.peek(r -> r.setState(ComponentLifecycleState.SUSPENDED))
.findFirst()
.orElse(null);
}
if (rule != null) {
RuleActorMetaData actorMd = ruleMap.get(rule);
@ -92,7 +93,7 @@ public abstract class RuleManager {
refreshRuleChain();
return Optional.of(actorMd.getActorRef());
} else {
logger.warn("[{}] Can't process unknown rule!", rule.getId());
log.warn("[{}] Can't process unknown rule!", ruleId);
return Optional.empty();
}
}
@ -100,13 +101,9 @@ public abstract class RuleManager {
abstract FetchFunction<RuleMetaData> getFetchRulesFunction();
public ActorRef getOrCreateRuleActor(ActorContext context, RuleId ruleId) {
ActorRef ruleActor = ruleActors.get(ruleId);
if (ruleActor == null) {
ruleActor = context.actorOf(Props.create(new RuleActor.ActorCreator(systemContext, tenantId, ruleId))
.withDispatcher(DefaultActorService.RULE_DISPATCHER_NAME), ruleId.toString());
ruleActors.put(ruleId, ruleActor);
}
return ruleActor;
return ruleActors.computeIfAbsent(ruleId, rId ->
context.actorOf(Props.create(new RuleActor.ActorCreator(systemContext, tenantId, rId))
.withDispatcher(DefaultActorService.RULE_DISPATCHER_NAME), rId.toString()));
}
public RuleActorChain getRuleChain() {

View File

@ -29,7 +29,7 @@ public class SystemRuleManager extends RuleManager {
@Override
FetchFunction<RuleMetaData> getFetchRulesFunction() {
return link -> ruleService.findSystemRules(link);
return ruleService::findSystemRules;
}
}

View File

@ -100,7 +100,7 @@ public class TenantActor extends ContextAwareActor {
private void broadcast(Object msg) {
pluginManager.broadcast(msg);
deviceActors.values().stream().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
deviceActors.values().forEach(actorRef -> actorRef.tell(msg, ActorRef.noSender()));
}
private void onToDeviceActorMsg(ToDeviceActorMsg msg) {

View File

@ -166,7 +166,7 @@ public class ZkDiscoveryService implements DiscoveryService, PathChildrenCacheLi
@Override
public void onApplicationEvent(ApplicationReadyEvent applicationReadyEvent) {
publishCurrentServer();
getOtherServers().stream().forEach(
getOtherServers().forEach(
server -> log.info("Found active server: [{}:{}]", server.getHost(), server.getPort())
);
}
@ -194,13 +194,13 @@ public class ZkDiscoveryService implements DiscoveryService, PathChildrenCacheLi
log.info("Processing [{}] event for [{}:{}]", pathChildrenCacheEvent.getType(), instance.getHost(), instance.getPort());
switch (pathChildrenCacheEvent.getType()) {
case CHILD_ADDED:
listeners.stream().forEach(listener -> listener.onServerAdded(instance));
listeners.forEach(listener -> listener.onServerAdded(instance));
break;
case CHILD_UPDATED:
listeners.stream().forEach(listener -> listener.onServerUpdated(instance));
listeners.forEach(listener -> listener.onServerUpdated(instance));
break;
case CHILD_REMOVED:
listeners.stream().forEach(listener -> listener.onServerRemoved(instance));
listeners.forEach(listener -> listener.onServerRemoved(instance));
break;
}
}

View File

@ -135,7 +135,7 @@ public class ConsistentClusterRoutingService implements ClusterRoutingService, D
private void logCircle() {
log.trace("Consistent Hash Circle Start");
circle.entrySet().stream().forEach((e) -> log.debug("{} -> {}", e.getKey(), e.getValue().getServerAddress()));
circle.entrySet().forEach((e) -> log.debug("{} -> {}", e.getKey(), e.getValue().getServerAddress()));
log.trace("Consistent Hash Circle End");
}

View File

@ -31,7 +31,6 @@ import org.thingsboard.server.dao.component.ComponentDescriptorService;
import org.thingsboard.server.extensions.api.component.*;
import javax.annotation.PostConstruct;
import java.io.IOException;
import java.lang.annotation.Annotation;
import java.util.*;
import java.util.stream.Collectors;
@ -72,7 +71,7 @@ public class AnnotationComponentDiscoveryService implements ComponentDiscoverySe
}
private void registerComponents(Collection<ComponentDescriptor> comps) {
comps.stream().forEach(c -> components.put(c.getClazz(), c));
comps.forEach(c -> components.put(c.getClazz(), c));
}
private List<ComponentDescriptor> persist(Set<BeanDefinition> filterDefs, ComponentType type) {
@ -119,7 +118,7 @@ public class AnnotationComponentDiscoveryService implements ComponentDiscoverySe
throw new RuntimeException("Plugin " + def.getBeanClassName() + "action " + actionClazz.getName() + " has wrong component type!");
}
}
scannedComponent.setActions(Arrays.asList(pluginAnnotation.actions()).stream().map(action -> action.getName()).collect(Collectors.joining(",")));
scannedComponent.setActions(Arrays.stream(pluginAnnotation.actions()).map(action -> action.getName()).collect(Collectors.joining(",")));
break;
default:
throw new RuntimeException(type + " is not supported yet!");

View File

@ -20,9 +20,9 @@ import org.springframework.security.core.authority.SimpleGrantedAuthority;
import org.thingsboard.server.common.data.User;
import org.thingsboard.server.common.data.id.UserId;
import java.util.Arrays;
import java.util.Collection;
import java.util.stream.Collectors;
import java.util.stream.Stream;
public class SecurityUser extends User {
@ -46,7 +46,7 @@ public class SecurityUser extends User {
public Collection<? extends GrantedAuthority> getAuthorities() {
if (authorities == null) {
authorities = Arrays.asList(SecurityUser.this.getAuthority()).stream()
authorities = Stream.of(SecurityUser.this.getAuthority())
.map(authority -> new SimpleGrantedAuthority(authority.name()))
.collect(Collectors.toList());
}

View File

@ -129,8 +129,10 @@ public abstract class AbstractControllerTest {
@Autowired
void setConverters(HttpMessageConverter<?>[] converters) {
this.mappingJackson2HttpMessageConverter = Arrays.asList(converters).stream().filter(
hmc -> hmc instanceof MappingJackson2HttpMessageConverter).findAny().get();
this.mappingJackson2HttpMessageConverter = Arrays.stream(converters)
.filter(hmc -> hmc instanceof MappingJackson2HttpMessageConverter)
.findAny()
.get();
Assert.assertNotNull("the JSON message converter must not be null",
this.mappingJackson2HttpMessageConverter);

View File

@ -61,8 +61,10 @@ public class AbstractFeatureIntegrationTest {
@Autowired
void setConverters(HttpMessageConverter<?>[] converters) {
this.mappingJackson2HttpMessageConverter = Arrays.asList(converters).stream().filter(
hmc -> hmc instanceof MappingJackson2HttpMessageConverter).findAny().get();
this.mappingJackson2HttpMessageConverter = Arrays.stream(converters)
.filter(hmc -> hmc instanceof MappingJackson2HttpMessageConverter)
.findAny()
.get();
assertNotNull("the JSON message converter must not be null",
this.mappingJackson2HttpMessageConverter);

View File

@ -140,7 +140,7 @@ public class BaseAttributesDao extends AbstractDao implements AttributesDao {
List<Row> rows = resultSet.all();
List<AttributeKvEntry> entries = new ArrayList<>(rows.size());
if (!rows.isEmpty()) {
rows.stream().forEach(row -> {
rows.forEach(row -> {
String key = row.getString(ModelConstants.ATTRIBUTE_KEY_COLUMN);
AttributeKvEntry kvEntry = convertResultToAttributesKvEntry(key, row);
if (kvEntry != null) {

View File

@ -143,7 +143,7 @@ public class BaseTimeseriesDao extends AbstractDao implements TimeseriesDao {
public List<TsKvEntry> convertResultToTsKvEntryList(List<Row> rows) {
List<TsKvEntry> entries = new ArrayList<>(rows.size());
if (!rows.isEmpty()) {
rows.stream().forEach(row -> {
rows.forEach(row -> {
TsKvEntry kvEntry = convertResultToTsKvEntry(row);
if (kvEntry != null) {
entries.add(kvEntry);

View File

@ -91,7 +91,7 @@ public class DefaultWebsocketMsgHandler implements WebsocketMsgHandler {
}
public void clear(PluginContext ctx) {
wsSessionsMap.values().stream().forEach(v -> {
wsSessionsMap.values().forEach(v -> {
try {
ctx.close(v.getSessionRef());
} catch (IOException e) {

View File

@ -40,7 +40,9 @@ public class MethodNameFilter extends SimpleRuleLifecycleComponent implements Ru
@Override
public void init(MethodNameFilterConfiguration configuration) {
methods = Arrays.asList(configuration.getMethodNames()).stream().map(m -> m.getName()).collect(Collectors.toSet());
methods = Arrays.stream(configuration.getMethodNames())
.map(m -> m.getName())
.collect(Collectors.toSet());
}
@Override

View File

@ -39,7 +39,7 @@ public class MsgTypeFilter extends SimpleRuleLifecycleComponent implements RuleF
@Override
public void init(MsgTypeFilterConfiguration configuration) {
msgTypes = Arrays.asList(configuration.getMessageTypes()).stream().map(type -> {
msgTypes = Arrays.stream(configuration.getMessageTypes()).map(type -> {
switch (type) {
case "GET_ATTRIBUTES":
return MsgType.GET_ATTRIBUTES_REQUEST;

View File

@ -75,7 +75,7 @@ public class MailPlugin extends AbstractPlugin<MailPluginConfiguration> implemen
if (configuration.getOtherProperties() != null) {
Properties mailProperties = new Properties();
configuration.getOtherProperties()
.stream().forEach(p -> mailProperties.put(p.getKey(), p.getValue()));
.forEach(p -> mailProperties.put(p.getKey(), p.getValue()));
mail.setJavaMailProperties(mailProperties);
}
mailSender = mail;

View File

@ -68,7 +68,7 @@ public class SubscriptionManager {
registerSubscription(sessionId, deviceId, subscription);
List<TsKvEntry> missedUpdates = new ArrayList<>();
if (subscription.getType() == SubscriptionType.ATTRIBUTES) {
subscription.getKeyStates().entrySet().stream().forEach(e -> {
subscription.getKeyStates().entrySet().forEach(e -> {
Optional<AttributeKvEntry> latestOpt = ctx.loadAttribute(deviceId, DataConstants.CLIENT_SCOPE, e.getKey());
if (latestOpt.isPresent()) {
AttributeKvEntry latestEntry = latestOpt.get();

View File

@ -97,7 +97,7 @@ public class TelemetryRpcMsgHandler implements RpcMsgHandler {
builder.setDeviceId(cmd.getDeviceId().toString());
builder.setType(cmd.getType().name());
builder.setAllKeys(cmd.isAllKeys());
cmd.getKeyStates().entrySet().stream().forEach(e -> builder.addKeyStates(SubscriptionKetStateProto.newBuilder().setKey(e.getKey()).setTs(e.getValue()).build()));
cmd.getKeyStates().entrySet().forEach(e -> builder.addKeyStates(SubscriptionKetStateProto.newBuilder().setKey(e.getKey()).setTs(e.getValue()).build()));
ctx.sendPluginRpcMsg(new RpcMsg(address, SUBSCRIPTION_CLAZZ, builder.build().toByteArray()));
}
@ -144,7 +144,7 @@ public class TelemetryRpcMsgHandler implements RpcMsgHandler {
if (update.getErrorMsg() != null) {
builder.setErrorMsg(update.getErrorMsg());
}
update.getData().entrySet().stream().forEach(
update.getData().entrySet().forEach(
e -> {
SubscriptionUpdateValueListProto.Builder dataBuilder = SubscriptionUpdateValueListProto.newBuilder();
@ -166,7 +166,7 @@ public class TelemetryRpcMsgHandler implements RpcMsgHandler {
return new SubscriptionUpdate(proto.getSubscriptionId(), SubscriptionErrorCode.forCode(proto.getErrorCode()), proto.getErrorMsg());
} else {
Map<String, List<Object>> data = new TreeMap<>();
proto.getDataList().stream().forEach(v -> {
proto.getDataList().forEach(v -> {
List<Object> values = data.get(v.getKey());
if (values == null) {
values = new ArrayList<>();

View File

@ -109,8 +109,8 @@ public class TelemetryWebsocketMsgHandler extends DefaultWebsocketMsgHandler {
sendWsMsg(ctx, sessionRef, new SubscriptionUpdate(cmd.getCmdId(), attributesData));
Map<String, Long> subState = new HashMap<>(keys.size());
keys.stream().forEach(key -> subState.put(key, 0L));
attributesData.stream().forEach(v -> subState.put(v.getKey(), v.getTs()));
keys.forEach(key -> subState.put(key, 0L));
attributesData.forEach(v -> subState.put(v.getKey(), v.getTs()));
sub = new SubscriptionState(sessionId, cmd.getCmdId(), deviceId, SubscriptionType.ATTRIBUTES, false, subState);
} else {
@ -119,7 +119,7 @@ public class TelemetryWebsocketMsgHandler extends DefaultWebsocketMsgHandler {
sendWsMsg(ctx, sessionRef, new SubscriptionUpdate(cmd.getCmdId(), attributesData));
Map<String, Long> subState = new HashMap<>(attributesData.size());
attributesData.stream().forEach(v -> subState.put(v.getKey(), v.getTs()));
attributesData.forEach(v -> subState.put(v.getKey(), v.getTs()));
sub = new SubscriptionState(sessionId, cmd.getCmdId(), deviceId, SubscriptionType.ATTRIBUTES, true, subState);
}
@ -154,8 +154,8 @@ public class TelemetryWebsocketMsgHandler extends DefaultWebsocketMsgHandler {
sendWsMsg(ctx, sessionRef, new SubscriptionUpdate(cmd.getCmdId(), data));
Map<String, Long> subState = new HashMap<>(keys.size());
keys.stream().forEach(key -> subState.put(key, startTs));
data.stream().forEach(v -> subState.put(v.getKey(), v.getTs()));
keys.forEach(key -> subState.put(key, startTs));
data.forEach(v -> subState.put(v.getKey(), v.getTs()));
SubscriptionState sub = new SubscriptionState(sessionId, cmd.getCmdId(), deviceId, SubscriptionType.TIMESERIES, false, subState);
subscriptionManager.addLocalWsSubscription(ctx, sessionId, deviceId, sub);
} else {
@ -168,8 +168,8 @@ public class TelemetryWebsocketMsgHandler extends DefaultWebsocketMsgHandler {
sendWsMsg(ctx, sessionRef, new SubscriptionUpdate(cmd.getCmdId(), data));
Map<String, Long> subState = new HashMap<>(keys.size());
keys.stream().forEach(key -> subState.put(key, startTs));
data.stream().forEach(v -> subState.put(v.getKey(), v.getTs()));
keys.forEach(key -> subState.put(key, startTs));
data.forEach(v -> subState.put(v.getKey(), v.getTs()));
SubscriptionState sub = new SubscriptionState(sessionId, cmd.getCmdId(), deviceId, SubscriptionType.TIMESERIES, false, subState);
subscriptionManager.addLocalWsSubscription(ctx, sessionId, deviceId, sub);
}
@ -188,7 +188,7 @@ public class TelemetryWebsocketMsgHandler extends DefaultWebsocketMsgHandler {
public void onSuccess(PluginContext ctx, List<TsKvEntry> data) {
sendWsMsg(ctx, sessionRef, new SubscriptionUpdate(cmd.getCmdId(), data));
Map<String, Long> subState = new HashMap<>(data.size());
data.stream().forEach(v -> subState.put(v.getKey(), v.getTs()));
data.forEach(v -> subState.put(v.getKey(), v.getTs()));
SubscriptionState sub = new SubscriptionState(sessionId, cmd.getCmdId(), deviceId, SubscriptionType.TIMESERIES, true, subState);
subscriptionManager.addLocalWsSubscription(ctx, sessionId, deviceId, sub);
}

View File

@ -47,7 +47,7 @@ public class KafkaPlugin extends AbstractPlugin<KafkaPluginConfiguration> {
properties.put("buffer.memory", configuration.getBufferMemory());
if (configuration.getOtherProperties() != null) {
configuration.getOtherProperties()
.stream().forEach(p -> properties.put(p.getKey(), p.getValue()));
.forEach(p -> properties.put(p.getKey(), p.getValue()));
}
init();
}