[jira] [Resolved] (SLING-5217) Verify if login failures to resource providers are handled correctly

2016-01-18 Thread Robert Munteanu (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5217?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Munteanu resolved SLING-5217.

Resolution: Fixed
  Assignee: Robert Munteanu

The code looks good to me and I've verified the behaviour using a MongoDB 
provider with the patches from SLING-5437.

Considering fixed.

> Verify if login failures to resource providers are handled correctly
> 
>
> Key: SLING-5217
> URL: https://issues.apache.org/jira/browse/SLING-5217
> Project: Sling
>  Issue Type: Sub-task
>  Components: ResourceResolver
>Reporter: Carsten Ziegeler
>Assignee: Robert Munteanu
> Fix For: Resource Resolver 1.3.0
>
>
> If a resource provider is marked as required auth, then the login is 
> performed on creation of the resource resolver impl - if it fails, not RR is 
> created. This is correct.
> For providers marked as lazy auth, if login fails they are not tried again 
> and they are treated as if auth was successful, but all get methods return 
> null and all set methods either throw or do nothing. I haven't verified this 
> in all places yet.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (SLING-5437) The NoSQL providers should throw LoginException if the connection to the NoSQL database can't be established

2016-01-18 Thread Robert Munteanu (JIRA)
Robert Munteanu created SLING-5437:
--

 Summary: The NoSQL providers should throw LoginException if the 
connection to the NoSQL database can't be established
 Key: SLING-5437
 URL: https://issues.apache.org/jira/browse/SLING-5437
 Project: Sling
  Issue Type: New Feature
  Components: NoSQL
Affects Versions: NoSQL MongoDB Resource Provider 1.0.0, NoSQL Couchbase 
Resource Provider 1.0.0, NoSQL Generic Resource Provider 1.0.0
Reporter: Robert Munteanu
Assignee: Robert Munteanu
 Fix For: NoSQL Generic Resource Provider 1.0.2, NoSQL Couchbase 
Client 1.0.2, NoSQL MongoDB Resource Provider 1.0.2


While trying to find a proper test for SLING-5217, I noticed that the NoSQL 
providers don't check the connection when the ResourceProvider is instantiated. 
Therefore the ResourceProvider is always considered valid and when access is 
attempted exceptions are thrown.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (SLING-5437) The NoSQL providers should throw LoginException if the connection to the NoSQL database can't be established

2016-01-18 Thread Robert Munteanu (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5437?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Munteanu updated SLING-5437:
---
Attachment: 0001-SLING-5437-The-NoSQL-providers-should-throw-LoginExc.patch

Added a rough patch which works for MongoDB ( couchbase is a no-op since I did 
not have the time to test it ).

I'm not very eager to commit it without review since it bumps the API of 
`org.apache.sling.nosql.generic.adapter` to 2.0.0 .

[~sseif...@pro-vision.de] - thoughts?

> The NoSQL providers should throw LoginException if the connection to the 
> NoSQL database can't be established
> 
>
> Key: SLING-5437
> URL: https://issues.apache.org/jira/browse/SLING-5437
> Project: Sling
>  Issue Type: New Feature
>  Components: NoSQL
>Affects Versions: NoSQL Generic Resource Provider 1.0.0, NoSQL Couchbase 
> Resource Provider 1.0.0, NoSQL MongoDB Resource Provider 1.0.0
>Reporter: Robert Munteanu
>Assignee: Robert Munteanu
> Fix For: NoSQL Generic Resource Provider 1.0.2, NoSQL Couchbase 
> Client 1.0.2, NoSQL MongoDB Resource Provider 1.0.2
>
> Attachments: 
> 0001-SLING-5437-The-NoSQL-providers-should-throw-LoginExc.patch
>
>
> While trying to find a proper test for SLING-5217, I noticed that the NoSQL 
> providers don't check the connection when the ResourceProvider is 
> instantiated. Therefore the ResourceProvider is always considered valid and 
> when access is attempted exceptions are thrown.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (SLING-5205) Check for potential memory leaks

2016-01-18 Thread Robert Munteanu (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-5205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15105445#comment-15105445
 ] 

Robert Munteanu commented on SLING-5205:


After changing the OSGi config of the MongoDB provider, the old duplicate is 
still retained. I've validated this with two heap dumps ( see 
[^duplicate-mongodb-resource-provider.png] ).

> Check for potential memory leaks
> 
>
> Key: SLING-5205
> URL: https://issues.apache.org/jira/browse/SLING-5205
> Project: Sling
>  Issue Type: Sub-task
>  Components: ResourceResolver
>Reporter: Carsten Ziegeler
> Fix For: Resource Resolver 1.3.0
>
> Attachments: duplicate-mongodb-resource-provider.png
>
>
> With long running resource resolvers and provider dynamically being 
> registered, we need to check that we don't hold too strong references on the 
> providers



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (SLING-5205) Check for potential memory leaks

2016-01-18 Thread Robert Munteanu (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Munteanu updated SLING-5205:
---
Attachment: duplicate-mongodb-resource-provider.png

> Check for potential memory leaks
> 
>
> Key: SLING-5205
> URL: https://issues.apache.org/jira/browse/SLING-5205
> Project: Sling
>  Issue Type: Sub-task
>  Components: ResourceResolver
>Reporter: Carsten Ziegeler
> Fix For: Resource Resolver 1.3.0
>
> Attachments: duplicate-mongodb-resource-provider.png
>
>
> With long running resource resolvers and provider dynamically being 
> registered, we need to check that we don't hold too strong references on the 
> providers



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (SLING-5205) Check for potential memory leaks

2016-01-18 Thread Robert Munteanu (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Munteanu updated SLING-5205:
---
Assignee: Robert Munteanu

> Check for potential memory leaks
> 
>
> Key: SLING-5205
> URL: https://issues.apache.org/jira/browse/SLING-5205
> Project: Sling
>  Issue Type: Sub-task
>  Components: ResourceResolver
>Reporter: Carsten Ziegeler
>Assignee: Robert Munteanu
> Fix For: Resource Resolver 1.3.0
>
> Attachments: duplicate-mongodb-resource-provider.png
>
>
> With long running resource resolvers and provider dynamically being 
> registered, we need to check that we don't hold too strong references on the 
> providers



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Comment Edited] (SLING-5205) Check for potential memory leaks

2016-01-18 Thread Robert Munteanu (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-5205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15105445#comment-15105445
 ] 

Robert Munteanu edited comment on SLING-5205 at 1/18/16 3:41 PM:
-

After changing the OSGi config of the MongoDB provider, the old provider is 
still retained. I've validated this with two heap dumps ( see 
[^duplicate-mongodb-resource-provider.png] ).


was (Author: rombert):
After changing the OSGi config of the MongoDB provider, the old duplicate is 
still retained. I've validated this with two heap dumps ( see 
[^duplicate-mongodb-resource-provider.png] ).

> Check for potential memory leaks
> 
>
> Key: SLING-5205
> URL: https://issues.apache.org/jira/browse/SLING-5205
> Project: Sling
>  Issue Type: Sub-task
>  Components: ResourceResolver
>Reporter: Carsten Ziegeler
> Fix For: Resource Resolver 1.3.0
>
> Attachments: duplicate-mongodb-resource-provider.png
>
>
> With long running resource resolvers and provider dynamically being 
> registered, we need to check that we don't hold too strong references on the 
> providers



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (SLING-5414) Make the contents of the provisioning model available at runtime

2016-01-18 Thread Bertrand Delacretaz (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-5414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15105105#comment-15105105
 ] 

Bertrand Delacretaz commented on SLING-5414:


I've tried an approach where the slingstart maven plugin generates a bundle 
with a "modelfragment" classifier, used by a provisioning model provider bundle 
to supply the model at runtime. It works but creates a "chicken and egg" 
dependency on the modelfragment bundle in launchpad/builder.

I have saved that variant at 
https://github.com/bdelacretaz/sling/tree/SLING-5414-wontwork but will need to 
change the strategy in the slingstart plugin - having that optionally inject 
the modelfragment bundle in the provisioning model, without having a dependency 
on it, should work.

> Make the contents of the provisioning model available at runtime
> 
>
> Key: SLING-5414
> URL: https://issues.apache.org/jira/browse/SLING-5414
> Project: Sling
>  Issue Type: New Feature
>  Components: Maven Plugins and Archetypes
>Affects Versions: Slingstart Maven Plugin 1.4.0
>Reporter: Bertrand Delacretaz
>Assignee: Bertrand Delacretaz
>Priority: Minor
> Fix For: Slingstart Maven Plugin 1.4.2
>
> Attachments: SLING-5414.patch
>
>
> For SLING-5355 we need to make additional sections from the provisioning 
> model available at runtime, so that ACLs can be set at startup, and also to 
> be able to reuse their definitions later for auditing/verification purposes.
> A simple way to do that is to copy those sections in the sling_bootstrap.txt 
> file. I have created a patch that does that, will attach it here for review.
> _edit: as discussed below, we'll make the full text of the model available at 
> runtime_



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (SLING-3891) Document Sling Content Distribution

2016-01-18 Thread Konrad Windszus (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15105075#comment-15105075
 ] 

Konrad Windszus commented on SLING-3891:


[~teofili] At first glance I couldn't find any documentation on the website. 
Could you add a link in 
http://sling.staging.apache.org/documentation/bundles.html in the section 
content?

> Document Sling Content Distribution
> ---
>
> Key: SLING-3891
> URL: https://issues.apache.org/jira/browse/SLING-3891
> Project: Sling
>  Issue Type: Task
>  Components: Distribution, Documentation
>Reporter: Tommaso Teofili
>Assignee: Tommaso Teofili
>
> Sling replication should be documented (APIs, endpoints, commands, etc.) to 
> ease usage and adoption.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (SLING-5358) Description for OSGi configuration of org.apache.sling.servlets.resolver.SlingServletResolver servletresolver.servletRoot is wrong

2016-01-18 Thread Konrad Windszus (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5358?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Konrad Windszus updated SLING-5358:
---
Fix Version/s: Servlets Resolver 2.3.10

> Description for OSGi configuration of 
> org.apache.sling.servlets.resolver.SlingServletResolver 
> servletresolver.servletRoot is wrong
> --
>
> Key: SLING-5358
> URL: https://issues.apache.org/jira/browse/SLING-5358
> Project: Sling
>  Issue Type: Bug
>  Components: Servlets
>Affects Versions: Servlets Resolver 2.3.8
>Reporter: Konrad Windszus
>Priority: Minor
> Fix For: Servlets Resolver 2.3.10
>
>
> The description of that property of the OSGi configuration says:
> {quote}
> The default root path assumed when registering a Servlet whose Servlet 
> registration properties define a relative path. The default value is "/apps". 
> This path should be part of the search path configured in the Resource 
> Resolver Factory otherwise a thus registered servlet may not be found.
> {quote}
> https://github.com/apache/sling/blob/trunk/bundles/servlets/resolver/src/main/resources/OSGI-INF/metatype/metatype.properties#L36
> The default value is actually 0 not "/apps" 
> (https://github.com/apache/sling/blob/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java#L137).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (SLING-5358) Description for OSGi configuration of org.apache.sling.servlets.resolver.SlingServletResolver servletresolver.servletRoot is wrong

2016-01-18 Thread Konrad Windszus (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5358?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Konrad Windszus updated SLING-5358:
---
Affects Version/s: (was: Servlets Resolver 2.3.10)
   Servlets Resolver 2.3.8

> Description for OSGi configuration of 
> org.apache.sling.servlets.resolver.SlingServletResolver 
> servletresolver.servletRoot is wrong
> --
>
> Key: SLING-5358
> URL: https://issues.apache.org/jira/browse/SLING-5358
> Project: Sling
>  Issue Type: Bug
>  Components: Servlets
>Affects Versions: Servlets Resolver 2.3.8
>Reporter: Konrad Windszus
>Priority: Minor
> Fix For: Servlets Resolver 2.3.10
>
>
> The description of that property of the OSGi configuration says:
> {quote}
> The default root path assumed when registering a Servlet whose Servlet 
> registration properties define a relative path. The default value is "/apps". 
> This path should be part of the search path configured in the Resource 
> Resolver Factory otherwise a thus registered servlet may not be found.
> {quote}
> https://github.com/apache/sling/blob/trunk/bundles/servlets/resolver/src/main/resources/OSGI-INF/metatype/metatype.properties#L36
> The default value is actually 0 not "/apps" 
> (https://github.com/apache/sling/blob/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java#L137).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (SLING-3891) Document Sling Content Distribution

2016-01-18 Thread Tommaso Teofili (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-3891?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tommaso Teofili resolved SLING-3891.

Resolution: Fixed

> Document Sling Content Distribution
> ---
>
> Key: SLING-3891
> URL: https://issues.apache.org/jira/browse/SLING-3891
> Project: Sling
>  Issue Type: Task
>  Components: Distribution, Documentation
>Reporter: Tommaso Teofili
>Assignee: Tommaso Teofili
>
> Sling replication should be documented (APIs, endpoints, commands, etc.) to 
> ease usage and adoption.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (SLING-5358) Description for OSGi configuration of org.apache.sling.servlets.resolver.SlingServletResolver servletresolver.servletRoot is wrong

2016-01-18 Thread Konrad Windszus (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5358?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Konrad Windszus resolved SLING-5358.

Resolution: Fixed

Fixed in [r1725219|http://svn.apache.org/r1725219].

> Description for OSGi configuration of 
> org.apache.sling.servlets.resolver.SlingServletResolver 
> servletresolver.servletRoot is wrong
> --
>
> Key: SLING-5358
> URL: https://issues.apache.org/jira/browse/SLING-5358
> Project: Sling
>  Issue Type: Bug
>  Components: Servlets
>Affects Versions: Servlets Resolver 2.3.8
>Reporter: Konrad Windszus
>Priority: Minor
> Fix For: Servlets Resolver 2.3.10
>
>
> The description of that property of the OSGi configuration says:
> {quote}
> The default root path assumed when registering a Servlet whose Servlet 
> registration properties define a relative path. The default value is "/apps". 
> This path should be part of the search path configured in the Resource 
> Resolver Factory otherwise a thus registered servlet may not be found.
> {quote}
> https://github.com/apache/sling/blob/trunk/bundles/servlets/resolver/src/main/resources/OSGI-INF/metatype/metatype.properties#L36
> The default value is actually 0 not "/apps" 
> (https://github.com/apache/sling/blob/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java#L137).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (SLING-4183) Jcr Installer Provider only supports storing array values for OSGi Component Properties (for sling:OsgiConfig resources) but not java.util.Vector

2016-01-18 Thread Konrad Windszus (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-4183?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Konrad Windszus updated SLING-4183:
---
Summary: Jcr Installer Provider only supports storing array values for OSGi 
Component Properties (for sling:OsgiConfig resources) but not java.util.Vector  
(was: Jcr Installer Provider only supports storing array values for OSGi 
Component Properties but not java.util.Vector)

> Jcr Installer Provider only supports storing array values for OSGi Component 
> Properties (for sling:OsgiConfig resources) but not java.util.Vector
> -
>
> Key: SLING-4183
> URL: https://issues.apache.org/jira/browse/SLING-4183
> Project: Sling
>  Issue Type: Improvement
>  Components: Installer
>Affects Versions: JCR Installer 3.1.8
>Reporter: Konrad Windszus
>
> The OSGi Compendium 4.3.0, $105.14.3.13 defines two storage formats for 
> multivalue entries: one is vector and the other one is array.
> As long as a configuration is maintained via the Felix Webconsole, it will 
> stick to the data format being given in the metatype manifest (through the 
> cardinality attribute).
> If the JCR Installer is used to deploy the OSGi configuration it will always 
> use array, in case the property is a multivalue property in the JCR 
> (https://github.com/apache/sling/blob/trunk/installer/providers/jcr/src/main/java/org/apache/sling/installer/provider/jcr/impl/ConfigNodeConverter.java#L99)
> In the best case the JCR Installer should evaluate the metatype manifest as 
> well. Or it should support {{Vector}}s through some special prefix on a 
> multivalue property name.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


SLING-5127 - ResourceProviderInfo.authType values clarification

2016-01-18 Thread Robert Munteanu
Hi,

I was looking at

  SLING-5217: Verify if login failures to resource providers are
handled correctly

and I tried to follow the logic behind the AuthType enum:

  public enum AuthType { no, lazy, required }

The 'lazy' and 'required' values don't really 'partition' the providers
well IMO, e.g. you can think of a resource provider that is required
and should be lazily authenticated against.

I would suggest the following change

  public enum AuthType { none, lazy, eager }

Thoughts?

Robert


[jira] [Resolved] (SLING-3553) Run FindBugs (and fix accordingly) on Sling Replication code

2016-01-18 Thread Tommaso Teofili (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-3553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tommaso Teofili resolved SLING-3553.

   Resolution: Fixed
Fix Version/s: Content Distribution Core 0.1.14

> Run FindBugs (and fix accordingly) on Sling Replication code
> 
>
> Key: SLING-3553
> URL: https://issues.apache.org/jira/browse/SLING-3553
> Project: Sling
>  Issue Type: Improvement
>  Components: Distribution
>Reporter: Tommaso Teofili
>Assignee: Tommaso Teofili
>Priority: Trivial
> Fix For: Content Distribution Core 0.1.14
>
>
> Sling Replication code has grown over time and I'd like to a) fix current 
> problems as reported by FindBugs b) add a reporting entry to the pom.xml in 
> order to be continuously able to check eventual problems.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (SLING-5436) Do not move distribution packages to error queues after connection problems

2016-01-18 Thread Marius Petria (JIRA)
Marius Petria created SLING-5436:


 Summary: Do not move distribution packages to error queues after 
connection problems
 Key: SLING-5436
 URL: https://issues.apache.org/jira/browse/SLING-5436
 Project: Sling
  Issue Type: Bug
  Components: Distribution
Reporter: Marius Petria
Assignee: Marius Petria
 Fix For: Content Distribution Core 0.1.14






--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (SLING-5439) HttpEntity in SimpleHttpTransport should be consumed / discarded

2016-01-18 Thread Tommaso Teofili (JIRA)
Tommaso Teofili created SLING-5439:
--

 Summary: HttpEntity in SimpleHttpTransport should be consumed / 
discarded
 Key: SLING-5439
 URL: https://issues.apache.org/jira/browse/SLING-5439
 Project: Sling
  Issue Type: Improvement
  Components: Distribution
Reporter: Tommaso Teofili
Assignee: Tommaso Teofili
 Fix For: Content Distribution Core 0.1.14


{{SimpleHttpDistributionTransport#deliverPackage}} returns Http {{Content}} but 
doesn't use it. While it seems it's not possible to just let it go (as that 
would leak as an unconsumed resource) such content should be discarded.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (SLING-5436) Do not move distribution packages to error queues after connection problems

2016-01-18 Thread Marius Petria (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Marius Petria resolved SLING-5436.
--
Resolution: Fixed

> Do not move distribution packages to error queues after connection problems
> ---
>
> Key: SLING-5436
> URL: https://issues.apache.org/jira/browse/SLING-5436
> Project: Sling
>  Issue Type: Bug
>  Components: Distribution
>Reporter: Marius Petria
>Assignee: Marius Petria
> Fix For: Content Distribution Core 0.1.14
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (SLING-5438) Avoid reopening the JcrPackage during installation

2016-01-18 Thread Tommaso Teofili (JIRA)
Tommaso Teofili created SLING-5438:
--

 Summary: Avoid reopening the JcrPackage during installation
 Key: SLING-5438
 URL: https://issues.apache.org/jira/browse/SLING-5438
 Project: Sling
  Issue Type: Improvement
  Components: Distribution
Reporter: Tommaso Teofili
Assignee: Tommaso Teofili
 Fix For: Content Distribution Core 0.1.14


During {{JcrVaultDistributionPackageBuilder#installPackageInternal}} the 
{{JcrPackage}} contained in the {{DistributionPackage}} is re opened using its 
id. That is avoidable as the {{JcrPackage}} is already available inside the 
{{DistributionPackage}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (SLING-5438) Avoid reopening the JcrPackage during installation

2016-01-18 Thread Tommaso Teofili (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5438?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tommaso Teofili resolved SLING-5438.

Resolution: Fixed

fixed in r1725302

> Avoid reopening the JcrPackage during installation
> --
>
> Key: SLING-5438
> URL: https://issues.apache.org/jira/browse/SLING-5438
> Project: Sling
>  Issue Type: Improvement
>  Components: Distribution
>Reporter: Tommaso Teofili
>Assignee: Tommaso Teofili
> Fix For: Content Distribution Core 0.1.14
>
>
> During {{JcrVaultDistributionPackageBuilder#installPackageInternal}} the 
> {{JcrPackage}} contained in the {{DistributionPackage}} is re opened using 
> its id. That is avoidable as the {{JcrPackage}} is already available inside 
> the {{DistributionPackage}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (SLING-5205) Check for potential memory leaks

2016-01-18 Thread Robert Munteanu (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-5205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15105720#comment-15105720
 ] 

Robert Munteanu commented on SLING-5205:


For the sake of being complete, here's a screenshot of the path to the GC root 
for the leaking resource provider [^shortest-path.png]

> Check for potential memory leaks
> 
>
> Key: SLING-5205
> URL: https://issues.apache.org/jira/browse/SLING-5205
> Project: Sling
>  Issue Type: Sub-task
>  Components: ResourceResolver
>Reporter: Carsten Ziegeler
>Assignee: Robert Munteanu
> Fix For: Resource Resolver 1.3.0
>
> Attachments: duplicate-mongodb-resource-provider.png, 
> shortest-path.png
>
>
> With long running resource resolvers and provider dynamically being 
> registered, we need to check that we don't hold too strong references on the 
> providers



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (SLING-5205) Check for potential memory leaks

2016-01-18 Thread Robert Munteanu (JIRA)

 [ 
https://issues.apache.org/jira/browse/SLING-5205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Munteanu updated SLING-5205:
---
Attachment: shortest-path.png

> Check for potential memory leaks
> 
>
> Key: SLING-5205
> URL: https://issues.apache.org/jira/browse/SLING-5205
> Project: Sling
>  Issue Type: Sub-task
>  Components: ResourceResolver
>Reporter: Carsten Ziegeler
>Assignee: Robert Munteanu
> Fix For: Resource Resolver 1.3.0
>
> Attachments: duplicate-mongodb-resource-provider.png, 
> shortest-path.png
>
>
> With long running resource resolvers and provider dynamically being 
> registered, we need to check that we don't hold too strong references on the 
> providers



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)