[Minor] Log how long persistence of users takes
This commit is contained in:
parent
d46bf18fa2
commit
b6d1c045ae
|
@ -15,16 +15,11 @@
|
||||||
*/
|
*/
|
||||||
package li.strolch.privilege.handler;
|
package li.strolch.privilege.handler;
|
||||||
|
|
||||||
|
import static li.strolch.utils.helper.StringHelper.formatNanoDuration;
|
||||||
|
|
||||||
import java.io.File;
|
import java.io.File;
|
||||||
import java.text.MessageFormat;
|
import java.text.MessageFormat;
|
||||||
import java.util.Collections;
|
import java.util.*;
|
||||||
import java.util.HashMap;
|
|
||||||
import java.util.LinkedList;
|
|
||||||
import java.util.List;
|
|
||||||
import java.util.Map;
|
|
||||||
|
|
||||||
import org.slf4j.Logger;
|
|
||||||
import org.slf4j.LoggerFactory;
|
|
||||||
|
|
||||||
import li.strolch.privilege.base.PrivilegeException;
|
import li.strolch.privilege.base.PrivilegeException;
|
||||||
import li.strolch.privilege.helper.XmlConstants;
|
import li.strolch.privilege.helper.XmlConstants;
|
||||||
|
@ -36,6 +31,8 @@ import li.strolch.privilege.xml.PrivilegeUsersDomWriter;
|
||||||
import li.strolch.privilege.xml.PrivilegeUsersSaxReader;
|
import li.strolch.privilege.xml.PrivilegeUsersSaxReader;
|
||||||
import li.strolch.utils.helper.StringHelper;
|
import li.strolch.utils.helper.StringHelper;
|
||||||
import li.strolch.utils.helper.XmlHelper;
|
import li.strolch.utils.helper.XmlHelper;
|
||||||
|
import org.slf4j.Logger;
|
||||||
|
import org.slf4j.LoggerFactory;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* {@link PersistenceHandler} implementation which reads the configuration from XML files. These configuration is passed
|
* {@link PersistenceHandler} implementation which reads the configuration from XML files. These configuration is passed
|
||||||
|
@ -136,7 +133,8 @@ public class XmlPersistenceHandler implements PersistenceHandler {
|
||||||
* Initializes this {@link XmlPersistenceHandler} by reading the following parameters:
|
* Initializes this {@link XmlPersistenceHandler} by reading the following parameters:
|
||||||
* <ul>
|
* <ul>
|
||||||
* <li>{@link XmlConstants#XML_PARAM_BASE_PATH}</li>
|
* <li>{@link XmlConstants#XML_PARAM_BASE_PATH}</li>
|
||||||
* <li>{@link XmlConstants#XML_PARAM_MODEL_FILE}</li>
|
* <li>{@link XmlConstants#XML_PARAM_USERS_FILE}</li>
|
||||||
|
* <li>{@link XmlConstants#XML_PARAM_ROLES_FILE}</li>
|
||||||
* </ul>
|
* </ul>
|
||||||
*/
|
*/
|
||||||
@Override
|
@Override
|
||||||
|
@ -261,6 +259,8 @@ public class XmlPersistenceHandler implements PersistenceHandler {
|
||||||
@Override
|
@Override
|
||||||
public boolean persist() {
|
public boolean persist() {
|
||||||
|
|
||||||
|
long start = System.nanoTime();
|
||||||
|
|
||||||
// get users file name
|
// get users file name
|
||||||
String usersFileName = this.parameterMap.get(XmlConstants.XML_PARAM_USERS_FILE);
|
String usersFileName = this.parameterMap.get(XmlConstants.XML_PARAM_USERS_FILE);
|
||||||
if (usersFileName == null || usersFileName.isEmpty()) {
|
if (usersFileName == null || usersFileName.isEmpty()) {
|
||||||
|
@ -282,7 +282,8 @@ public class XmlPersistenceHandler implements PersistenceHandler {
|
||||||
// get users file
|
// get users file
|
||||||
boolean usersFileUnchanged = this.usersPath.exists() && this.usersPath.lastModified() == this.usersFileDate;
|
boolean usersFileUnchanged = this.usersPath.exists() && this.usersPath.lastModified() == this.usersFileDate;
|
||||||
if (usersFileUnchanged && !this.userMapDirty) {
|
if (usersFileUnchanged && !this.userMapDirty) {
|
||||||
logger.warn("Not persisting of users as current file is unchanged and users data is not dirty"); //$NON-NLS-1$
|
logger.warn(
|
||||||
|
"Not persisting of users as current file is unchanged and users data is not dirty"); //$NON-NLS-1$
|
||||||
} else {
|
} else {
|
||||||
|
|
||||||
// delegate writing
|
// delegate writing
|
||||||
|
@ -296,7 +297,8 @@ public class XmlPersistenceHandler implements PersistenceHandler {
|
||||||
// get roles file
|
// get roles file
|
||||||
boolean rolesFileUnchanged = this.rolesPath.exists() && this.rolesPath.lastModified() == this.rolesFileDate;
|
boolean rolesFileUnchanged = this.rolesPath.exists() && this.rolesPath.lastModified() == this.rolesFileDate;
|
||||||
if (rolesFileUnchanged && !this.roleMapDirty) {
|
if (rolesFileUnchanged && !this.roleMapDirty) {
|
||||||
logger.warn("Not persisting of roles as current file is unchanged and roles data is not dirty"); //$NON-NLS-1$
|
logger.warn(
|
||||||
|
"Not persisting of roles as current file is unchanged and roles data is not dirty"); //$NON-NLS-1$
|
||||||
} else {
|
} else {
|
||||||
|
|
||||||
// delegate writing
|
// delegate writing
|
||||||
|
@ -307,8 +309,7 @@ public class XmlPersistenceHandler implements PersistenceHandler {
|
||||||
saved = true;
|
saved = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
// reset dirty states
|
logger.info("Persist took " + (formatNanoDuration(System.nanoTime() - start)));
|
||||||
|
|
||||||
return saved;
|
return saved;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue