Change loadMode to be 'safer' (for Grafana etc). e.g. change "(+lazy)" to "__lazy"

The loadMode ends up as a suffix to some query metrics.

For example "foo.findIt_baz(+lazy)", and the (+lazy) part isn't safe/friendly to tools
like Grafana. So change to instead of (+lazy) use __lazy.
This commit is contained in:
robin.bygrave
2023-11-01 15:20:18 +13:00
parent 70eb557929
commit 8b0f58b3c2
4 changed files with 5 additions and 5 deletions
@@ -111,7 +111,7 @@ public final class LoadBeanRequest extends LoadRequest {
}
private String mode() {
return lazy ? "+lazy" : loadBuffer.isCache() ? "+cache" : "+query";
return lazy ? "lazy" : loadBuffer.isCache() ? "cache" : "query";
}
/**
@@ -101,7 +101,7 @@ public final class LoadManyRequest extends LoadRequest {
query.setLazyLoadForParents(many);
many.addWhereParentIdIn(query, parentIdList(server), loadContext.isUseDocStore());
query.setPersistenceContext(loadContext.persistenceContext());
query.setLoadDescription(lazy ? "+lazy" : "+query", description());
query.setLoadDescription(lazy ? "lazy" : "query", description());
if (lazy) {
query.setLazyLoadBatchSize(loadContext.batchSize());
} else {
@@ -82,9 +82,9 @@ final class DefaultBeanLoader {
// populate a new collection
BeanCollection<?> emptyCollection = many.createEmpty(parentBean);
many.setValue(parentBean, emptyCollection);
query.setLoadDescription("+refresh", null);
query.setLoadDescription("refresh", null);
} else {
query.setLoadDescription("+lazy", null);
query.setLoadDescription("lazy", null);
}
query.select(parentDesc.idBinder().idSelect());
@@ -52,7 +52,7 @@ abstract class DLoadBaseContext {
void setLabel(SpiQuery<?> query) {
String label = parent.planLabel();
if (label != null) {
query.setProfilePath(label, fullPath + "(" + query.loadMode() + ")", parent.profileLocation());
query.setProfilePath(label, fullPath + "__" + query.loadMode(), parent.profileLocation());
}
}