honeymoose
/
hapi-fhir
mirror of https://github.com/hapifhir/hapi-fhir.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Reduce search log noise

This commit is contained in:
James Agnew 2017-12-08 14:23:39 -05:00
parent 561dc86eb4
commit c86a2866d8
2 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
hapi-fhir-jpaserver-base/src/main/java/ca/uhn/fhir/jpa/dao/SearchBuilder.java
View File

@ -1528,7 +1528,7 @@ public class SearchBuilder implements ISearchBuilder {
thePredicates.add(joinParam1); thePredicates.add(joinParam1);
} }
} else { } else {
ourLog.info("Reusing join for {}", theSort.getParamName()); ourLog.debug("Reusing join for {}", theSort.getParamName());
} }
for (String next : sortAttrName) { for (String next : sortAttrName) {
@ -1981,7 +1981,7 @@ public class SearchBuilder implements ISearchBuilder {
List<Predicate> lastUpdatedPredicates = new ArrayList<Predicate>(); List<Predicate> lastUpdatedPredicates = new ArrayList<Predicate>();
if (theLastUpdated != null) { if (theLastUpdated != null) {
if (theLastUpdated.getLowerBoundAsInstant() != null) { if (theLastUpdated.getLowerBoundAsInstant() != null) {
ourLog.info("LastUpdated lower bound: {}", new InstantDt(theLastUpdated.getLowerBoundAsInstant())); ourLog.debug("LastUpdated lower bound: {}", new InstantDt(theLastUpdated.getLowerBoundAsInstant()));
Predicate predicateLower = builder.greaterThanOrEqualTo(from.<Date>get("myUpdated"), theLastUpdated.getLowerBoundAsInstant()); Predicate predicateLower = builder.greaterThanOrEqualTo(from.<Date>get("myUpdated"), theLastUpdated.getLowerBoundAsInstant());
lastUpdatedPredicates.add(predicateLower); lastUpdatedPredicates.add(predicateLower);
} }
@ -2203,12 +2203,12 @@ public class SearchBuilder implements ISearchBuilder {
} // if we need to fetch the next result } // if we need to fetch the next result
if (myFirst) { if (myFirst) {
ourLog.info("Initial query result returned in {}ms for query {}", myStopwatch.getMillis(), mySearchUuid); ourLog.debug("Initial query result returned in {}ms for query {}", myStopwatch.getMillis(), mySearchUuid);
myFirst = false; myFirst = false;
} }
if (myNext == NO_MORE) { if (myNext == NO_MORE) {
ourLog.info("Query found {} matches in {}ms for query {}", myPidSet.size(), myStopwatch.getMillis(), mySearchUuid); ourLog.debug("Query found {} matches in {}ms for query {}", myPidSet.size(), myStopwatch.getMillis(), mySearchUuid);
} }
} }
@ -2280,10 +2280,10 @@ public class SearchBuilder implements ISearchBuilder {
private void ensureHaveQuery() { private void ensureHaveQuery() {
if (myWrap == null) { if (myWrap == null) {
ourLog.info("Searching for unique index matches over {} candidate query strings", myUniqueQueryStrings.size()); ourLog.debug("Searching for unique index matches over {} candidate query strings", myUniqueQueryStrings.size());
StopWatch sw = new StopWatch(); StopWatch sw = new StopWatch();
Collection<Long> resourcePids = myCallingDao.getResourceIndexedCompositeStringUniqueDao().findResourcePidsByQueryStrings(myUniqueQueryStrings); Collection<Long> resourcePids = myCallingDao.getResourceIndexedCompositeStringUniqueDao().findResourcePidsByQueryStrings(myUniqueQueryStrings);
ourLog.info("Found {} unique index matches in {}ms", resourcePids.size(), sw.getMillis()); ourLog.debug("Found {} unique index matches in {}ms", resourcePids.size(), sw.getMillis());
myWrap = resourcePids.iterator(); myWrap = resourcePids.iterator();
} }
} }

4
src/changes/changes.xml
View File

@ -29,6 +29,10 @@
An unneccesary column called "MYHASHCODE" was added to the An unneccesary column called "MYHASHCODE" was added to the
HFJ_TAG_DEF table in the JPA server schema HFJ_TAG_DEF table in the JPA server schema
</action> </action>
<action type="fix">
A few log entries emitted by the JPA server suring every search have been reduced
from INFO to DEBUG in order to reduce log noise
</action>
</release> </release>
<release version="3.1.0" date="2017-11-23"> <release version="3.1.0" date="2017-11-23">
<action type="add"> <action type="add">