Skip to content

Commit 8e2a9bb

Browse files
authored
Merge pull request #420 from InseeFr/devVolum
fix: change query to have all questionnaireId
2 parents becb237 + 0f45c01 commit 8e2a9bb

4 files changed

Lines changed: 16 additions & 16 deletions

File tree

Dockerfile

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,14 @@ ENV PATH_TO_JAR=/opt/app/app.jar
44
WORKDIR /opt/app/
55
COPY ./target/*.jar $PATH_TO_JAR
66

7-
ENV JAVA_TOOL_OPTIONS_DEFAULT \
8-
-XX:MaxRAMPercentage=75
7+
ENV JAVA_TOOL_OPTIONS_DEFAULT="-XX:MaxRAMPercentage=75"
98

109
# Setup a non-root user context (security)
11-
RUN addgroup -g 1000 tomcatgroup \
12-
&& adduser -D -s / -u 1000 tomcatuser -G tomcatgroup \
13-
&& mkdir /opt/app/temp-files \
14-
&& chown -R 1000:1000 /opt/app
10+
RUN if ! getent passwd 1000 >/dev/null; then \
11+
useradd -u 1000 -g 1000 -s /usr/sbin/nologin -M tomcatuser; \
12+
fi \
13+
&& mkdir -p /opt/app/temp-files \
14+
&& chown -R 1000:1000 /opt/app
1515

1616
USER 1000
1717

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
<!-- Pi Test-->
3737
<pitest.version>1.22.1</pitest.version>
3838
<pitest.junit.version>1.2.3</pitest.junit.version>
39-
<bpm.version>1.0.26</bpm.version>
39+
<bpm.version>1.1.0</bpm.version>
4040
</properties>
4141
<dependencies>
4242
<dependency>

src/main/java/fr/insee/genesis/domain/service/volumetry/VolumetryLogService.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22

33
import fr.insee.genesis.Constants;
44
import fr.insee.genesis.configuration.Config;
5+
import fr.insee.genesis.domain.ports.api.LunaticJsonRawDataApiPort;
56
import fr.insee.genesis.domain.ports.api.RawResponseApiPort;
67
import fr.insee.genesis.domain.ports.api.SurveyUnitApiPort;
7-
import fr.insee.genesis.domain.ports.api.LunaticJsonRawDataApiPort;
88
import lombok.extern.slf4j.Slf4j;
99
import org.springframework.beans.factory.annotation.Autowired;
1010
import org.springframework.stereotype.Service;
@@ -98,7 +98,7 @@ public Map<String, Map<String, Long>> writeRawDataVolumetries(
9898

9999
Files.writeString(
100100
logFilePath,
101-
"questionnaireId;%s;%s;%s;distinctInterrogationIds\n"
101+
"questionnaireId;%s;%s;%s;distinctInterrogationIds%n"
102102
.formatted(
103103
Constants.MONGODB_LUNATIC_RAWDATA_COLLECTION_NAME,
104104
Constants.MONGODB_RAW_RESPONSES_COLLECTION_NAME,

src/main/java/fr/insee/genesis/infrastructure/adapter/LunaticJsonRawDataMongoAdapter.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -95,14 +95,14 @@ public void updateProcessDates(String campaignId, Set<String> interrogationIds)
9595

9696
@Override
9797
public Set<String> findDistinctQuestionnaireIds() {
98-
Set<String> questionnaireIds = new HashSet<>();
99-
for (String questionnaireId : mongoTemplate.getCollection(Constants.MONGODB_RAW_RESPONSES_COLLECTION_NAME).distinct(
100-
"questionnaireId",
101-
String.class)) {
102-
questionnaireIds.add(questionnaireId);
98+
List<String> ids = mongoTemplate.query(LunaticJsonRawDataDocument.class)
99+
.distinct("questionnaireId")
100+
.as(String.class)
101+
.all();
102+
return ids.stream()
103+
.filter(id -> id != null && !id.isBlank())
104+
.collect(java.util.stream.Collectors.toSet());
103105
}
104-
return questionnaireIds;
105-
}
106106

107107
@Override
108108
public Page<LunaticJsonRawDataModel> findByCampaignIdAndDate(String campaignId, Instant startDt, Instant endDt, Pageable pageable) {

0 commit comments

Comments
 (0)