[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-17 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15582014#comment-15582014 ] Stefan Egli commented on SLING-6070: Agreed, we shouldn't make any dependency on underlying order. So

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-17 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15581902#comment-15581902 ] Carsten Ziegeler commented on SLING-6070: - No, the maps can't be removed entirely. For example,

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-17 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15581779#comment-15581779 ] Stefan Egli commented on SLING-6070: My original idea for this improvement was to call

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-16 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15579730#comment-15579730 ] Carsten Ziegeler commented on SLING-6070: - I've optimized the memory handling a little bit,

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-11 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15565377#comment-15565377 ] Stefan Egli commented on SLING-6070: bq. Since Oak uses a breadth-first tree traversal when compiling

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-05 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15549579#comment-15549579 ] Carsten Ziegeler commented on SLING-6070: - Yes I think that makes sense > Reduce temporary

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-05 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15549108#comment-15549108 ] Stefan Egli commented on SLING-6070: should we even mark it as deprecated then? > Reduce temporary

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-05 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15549079#comment-15549079 ] Carsten Ziegeler commented on SLING-6070: - Yes, I think we don't need to support this for the

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-10-05 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15549068#comment-15549068 ] Stefan Egli commented on SLING-6070: [~cziegeler], re my question on OAK-4581: bq. If we no longer

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-09-29 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15532197#comment-15532197 ] Stefan Egli commented on SLING-6070: See

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-09-27 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15526129#comment-15526129 ] Stefan Egli commented on SLING-6070: Depending on the outcome of OAK-4853 this might become obsolete

[jira] [Commented] (SLING-6070) Reduce temporary storage required in JcrResourceListener, call reportChanges earlier

2016-09-27 Thread Stefan Egli (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15525986#comment-15525986 ] Stefan Egli commented on SLING-6070: Open question here is: are we allowed to make this assumption