Browse Source

refactored due to comments

pull/12191/head
YevhenBondarenko 2 years ago
parent
commit
dcf4eb2793
  1. 16
      common/queue/src/main/java/org/thingsboard/server/queue/usagestats/DefaultTbApiUsageReportClient.java

16
common/queue/src/main/java/org/thingsboard/server/queue/usagestats/DefaultTbApiUsageReportClient.java

@ -15,6 +15,7 @@
*/ */
package org.thingsboard.server.queue.usagestats; package org.thingsboard.server.queue.usagestats;
import com.google.common.collect.Lists;
import jakarta.annotation.PostConstruct; import jakarta.annotation.PostConstruct;
import lombok.Data; import lombok.Data;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
@ -52,7 +53,6 @@ import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.ConcurrentMap; import java.util.concurrent.ConcurrentMap;
import java.util.concurrent.TimeUnit; import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicLong; import java.util.concurrent.atomic.AtomicLong;
import java.util.stream.IntStream;
@Component @Component
@Slf4j @Slf4j
@ -155,13 +155,13 @@ public class DefaultTbApiUsageReportClient implements TbApiUsageReportClient {
} }
private List<ToUsageStatsServiceMsgPack> toMsgPack(List<ToUsageStatsServiceMsg> list) { private List<ToUsageStatsServiceMsgPack> toMsgPack(List<ToUsageStatsServiceMsg> list) {
return IntStream.range(0, (list.size() + packSize - 1) / packSize) return Lists.partition(list, packSize)
.mapToObj(i -> { .stream()
var packList = list.subList(i * packSize, Math.min((i + 1) * packSize, list.size())); .map(partition ->
var pack = ToUsageStatsServiceMsgPack.newBuilder(); ToUsageStatsServiceMsgPack.newBuilder()
pack.addAllMsgs(packList); .addAllMsgs(partition)
return pack.build(); .build())
}).toList(); .toList();
} }
@Override @Override

Loading…
Cancel
Save