Skip to content

Commit

Permalink
more clean-up of log info
Browse files Browse the repository at this point in the history
  • Loading branch information
qqmyers committed Sep 27, 2024
1 parent c912209 commit 821918f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions src/main/java/edu/harvard/iq/dataverse/DatasetPage.java
Original file line number Diff line number Diff line change
Expand Up @@ -2697,10 +2697,8 @@ public void edit(EditMode editMode) {
dataset = datasetService.find(dataset.getId());
}
workingVersion = dataset.getOrCreateEditVersion();
long start = System.currentTimeMillis();
if(!FeatureFlags.DISABLE_EDIT_DRAFT_LOGGING.enabled())
clone = workingVersion.cloneDatasetVersion();
logger.info("Cloning done at " + (System.currentTimeMillis()-start) + " ms");
if (editMode.equals(EditMode.METADATA)) {
datasetVersionUI = datasetVersionUI.initDatasetVersionUI(workingVersion, true);
updateDatasetFieldInputLevels();
Expand Down Expand Up @@ -3995,7 +3993,7 @@ public String save() {
long start = System.currentTimeMillis();

dataset = commandEngine.submit(cmd);
logger.info("All done in: " + (System.currentTimeMillis()-start) + " ms");
logger.fine("Save done in: " + (System.currentTimeMillis()-start) + " ms");
if (editMode == EditMode.CREATE) {
if (session.getUser() instanceof AuthenticatedUser) {
userNotificationService.sendNotification((AuthenticatedUser) session.getUser(), dataset.getCreateDate(), UserNotification.Type.CREATEDS, dataset.getLatestVersion().getId());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ public Dataset execute(CommandContext ctxt) throws CommandException {
AuthenticatedUser au = (AuthenticatedUser) getUser();
DatasetVersionDifference dvDifference = new DatasetVersionDifference(editVersion, clone);
ctxt.datasetVersion().writeEditVersionLog(dvDifference, au);
logger.info("log done at: " + (System.currentTimeMillis()-startTime));
logger.fine("log done at: " + (System.currentTimeMillis()-startTime));
}
} finally {
// We're done making changes - remove the lock...
Expand Down

0 comments on commit 821918f

Please sign in to comment.