[OAK-10706] Session.save() should check first if there are pending changes

• Type: Task
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.60.0
• Component/s: jcr
• Assignee: Unassigned
• Reporter: Joerg Hoh
• Created: 11/Mar/24 09:57
• Updated: 11/Mar/24 09:57

Description

Currently Session.save() works under the assumption that there are changes to be committed. That means there is some static runtime overhead, even if no changes are to be committed.

For that reason it's common to use this pattern to avoid this operation if nothing is to be saved.

if (session.hasPendingChanges()) {
  session.save();
}

But it would be really great if Oak would do this check internally, and avoid the overhead if there's nothing to save at all.