From 64618da6a71f0d6f65bbfe80ab0a73c950bd85e1 Mon Sep 17 00:00:00 2001 From: Robin Bygrave Date: Thu, 21 Jan 2016 17:52:23 +1300 Subject: [PATCH] #534 - REFACTOR: Internals for lazy loading specifically improving errors/logging in cases of lazy load errors due to deletes - Update debug logging for LoadManyRequest when empty collection found (expected frequently) --- .../java/com/avaje/ebeaninternal/api/LoadManyRequest.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/avaje/ebeaninternal/api/LoadManyRequest.java b/src/main/java/com/avaje/ebeaninternal/api/LoadManyRequest.java index 412a49109..25af86df3 100644 --- a/src/main/java/com/avaje/ebeaninternal/api/LoadManyRequest.java +++ b/src/main/java/com/avaje/ebeaninternal/api/LoadManyRequest.java @@ -2,6 +2,7 @@ package com.avaje.ebeaninternal.api; import com.avaje.ebean.EbeanServer; import com.avaje.ebean.bean.BeanCollection; +import com.avaje.ebean.bean.EntityBean; import com.avaje.ebeaninternal.server.core.OrmQueryRequest; import com.avaje.ebeaninternal.server.deploy.BeanDescriptor; import com.avaje.ebeaninternal.server.deploy.BeanPropertyAssocMany; @@ -17,7 +18,7 @@ import java.util.List; */ public class LoadManyRequest extends LoadRequest { - private static final Logger logger = LoggerFactory.getLogger(EbeanServer.class); + private static final Logger logger = LoggerFactory.getLogger(LoadManyRequest.class); private final List> batch; @@ -180,7 +181,9 @@ public class LoadManyRequest extends LoadRequest { BeanCollection bc = batch.get(i); if (bc.checkEmptyLazyLoad()) { if (logger.isDebugEnabled()) { - logger.debug("BeanCollection after load was empty. Owner:" + batch.get(i).getOwnerBean()); + EntityBean ownerBean = bc.getOwnerBean(); + Object parentId = desc.getId(ownerBean); + logger.debug("BeanCollection after lazy load was empty. type[" + ownerBean.getClass() + "] id[" + parentId + "] owner[" + ownerBean + "]"); } } else if (isLoadCache()) { Object parentId = desc.getId(bc.getOwnerBean());