Browse Source

Fixes after self-review before create WIP PR

pull/13920/head
dshvaika 1 year ago
parent
commit
ef9ad6751c
  1. 2
      application/src/main/java/org/thingsboard/server/actors/calculatedField/CalculatedFieldEntityMessageProcessor.java
  2. 3
      application/src/main/resources/logback.xml
  3. 8
      common/data/src/main/java/org/thingsboard/server/common/data/cf/configuration/CfArgumentDynamicSourceConfiguration.java
  4. 2
      common/data/src/main/java/org/thingsboard/server/common/data/cf/configuration/RelationQueryDynamicSourceConfiguration.java
  5. 2
      common/proto/src/main/proto/queue.proto

2
application/src/main/java/org/thingsboard/server/actors/calculatedField/CalculatedFieldEntityMessageProcessor.java

@ -302,8 +302,8 @@ public class CalculatedFieldEntityMessageProcessor extends AbstractContextAwareM
state = stateFuture.get(1, TimeUnit.MINUTES);
state.checkStateSize(new CalculatedFieldEntityCtxId(tenantId, ctx.getCfId(), entityId), ctx.getMaxStateSize());
states.put(ctx.getCfId(), state);
return state;
}
return state;
}
private void processStateIfReady(CalculatedFieldCtx ctx, List<CalculatedFieldId> cfIdList, CalculatedFieldState state, UUID tbMsgId, TbMsgType tbMsgType, TbCallback callback) throws CalculatedFieldException {

3
application/src/main/resources/logback.xml

@ -56,7 +56,8 @@
<!-- Device actor message processor debug -->
<!-- <logger name="org.thingsboard.server.actors.device.DeviceActorMessageProcessor" level="DEBUG" />-->
<logger name="org.thingsboard.server.actors.calculatedField" level="TRACE" />
<!-- CF actors message processors trace -->
<!-- <logger name="org.thingsboard.server.actors.calculatedField" level="TRACE" />-->
<logger name="com.microsoft.azure.servicebus.primitives.CoreMessageReceiver" level="OFF" />
<root level="INFO">

8
common/data/src/main/java/org/thingsboard/server/common/data/cf/configuration/CfArgumentDynamicSourceConfiguration.java

@ -19,8 +19,6 @@ import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
import com.fasterxml.jackson.annotation.JsonSubTypes;
import com.fasterxml.jackson.annotation.JsonTypeInfo;
import org.thingsboard.server.common.data.id.EntityId;
import org.thingsboard.server.common.data.relation.EntityRelationsQuery;
@JsonTypeInfo(
use = JsonTypeInfo.Id.NAME,
@ -38,10 +36,4 @@ public interface CfArgumentDynamicSourceConfiguration {
default void validate() {}
@JsonIgnore
boolean isSimpleRelation();
@JsonIgnore
EntityRelationsQuery toEntityRelationsQuery(EntityId rootEntityId);
}

2
common/data/src/main/java/org/thingsboard/server/common/data/cf/configuration/RelationQueryDynamicSourceConfiguration.java

@ -55,12 +55,10 @@ public class RelationQueryDynamicSourceConfiguration implements CfArgumentDynami
}
}
@Override
public boolean isSimpleRelation() {
return maxLevel == 1 && CollectionsUtil.isEmpty(entityTypes);
}
@Override
public EntityRelationsQuery toEntityRelationsQuery(EntityId rootEntityId) {
if (isSimpleRelation()) {
throw new IllegalArgumentException("Entity relations query can't be created for a simple relation!");

2
common/proto/src/main/proto/queue.proto

@ -910,7 +910,7 @@ message GeofencingZoneProto {
message GeofencingArgumentProto {
string argName = 1;
repeated GeofencingZoneProto zones = 4;
repeated GeofencingZoneProto zones = 2;
}
message CalculatedFieldStateProto {

Loading…
Cancel
Save