[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-24 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14906236#comment-14906236 ] Francesco Mari commented on OAK-3201: - Backported to 1.0 in r1705037. > Use static refe

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-24 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14906023#comment-14906023 ] Francesco Mari commented on OAK-3201: - Backported to 1.2 in r1705014. > Use static refe

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-18 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14805138#comment-14805138 ] Francesco Mari commented on OAK-3201: - Committed in trunk in r1703758. > Use static ref

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-17 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14791761#comment-14791761 ] Francesco Mari commented on OAK-3201: - bq. Yup but I do not see how that is related to c

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-17 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14791725#comment-14791725 ] Chetan Mehrotra commented on OAK-3201: -- bq. This may work the first time, but it doesn'

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-17 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14791710#comment-14791710 ] Francesco Mari commented on OAK-3201: - bq. Setting immediate to true - That should not b

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-16 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14791662#comment-14791662 ] Chetan Mehrotra commented on OAK-3201: -- Looks better!. Some more comments * Setting imm

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-15 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14745497#comment-14745497 ] Francesco Mari commented on OAK-3201: - I'm also wondering if explicit synchronization co

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-15 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14745023#comment-14745023 ] Chetan Mehrotra commented on OAK-3201: -- Missed reviewing it due to other issues. Some q

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-15 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14744985#comment-14744985 ] Francesco Mari commented on OAK-3201: - I'm pretty satisfied with the result, I'm going t

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-11 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14740568#comment-14740568 ] Chetan Mehrotra commented on OAK-3201: -- Approach look fine now. bq. I found very diffi

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736789#comment-14736789 ] Francesco Mari commented on OAK-3201: - [~dsuess], thanks for the pointer. I will try to

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread JIRA
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736730#comment-14736730 ] Dominik Süß commented on OAK-3201: -- [~frm], [~chetanm] - Sling hit a simliar issue when int

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736579#comment-14736579 ] Chetan Mehrotra commented on OAK-3201: -- bq. Every deployment using the current version

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736477#comment-14736477 ] Francesco Mari commented on OAK-3201: - Every deployment using the current version of {{S

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736394#comment-14736394 ] Chetan Mehrotra commented on OAK-3201: -- bq. Not true. They can still do it, but they ha

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736349#comment-14736349 ] Francesco Mari commented on OAK-3201: - bq. If we go with your fix then customer making u

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736341#comment-14736341 ] Chetan Mehrotra commented on OAK-3201: -- I would not be comfortable with this fix. We ca

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-09 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736335#comment-14736335 ] Francesco Mari commented on OAK-3201: - [~chetanm], continuing the offline conversation w

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-08 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14734953#comment-14734953 ] Chetan Mehrotra commented on OAK-3201: -- bq. Personally, I think that cardinality.minimu

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-08 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14734718#comment-14734718 ] Francesco Mari commented on OAK-3201: - [~chetanm], I'm aware of the change of behaviour.

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-08 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14734670#comment-14734670 ] Chetan Mehrotra commented on OAK-3201: -- [~frm] Can you provide some details on what is

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-08 Thread Marcel Reutegger (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14734543#comment-14734543 ] Marcel Reutegger commented on OAK-3201: --- With the most recent patch, the oak-pojosr te

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-07 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14733666#comment-14733666 ] Francesco Mari commented on OAK-3201: - bq. I still think whatever approach we take we sh

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-07 Thread Julian Reschke (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14733618#comment-14733618 ] Julian Reschke commented on OAK-3201: - It fails for me as well, and I just wasted hours

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-07 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14733563#comment-14733563 ] Chetan Mehrotra commented on OAK-3201: -- Missed on reading your first comment bq. This

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-07 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14733561#comment-14733561 ] Chetan Mehrotra commented on OAK-3201: -- [~frm] I looked into the test failure and it ap

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-07 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14733509#comment-14733509 ] Francesco Mari commented on OAK-3201: - bq. Please set a fix version when you resolve an

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-09-04 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14731031#comment-14731031 ] Francesco Mari commented on OAK-3201: - [~anchela], I agree with you with the idea that s

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-08-13 Thread angela (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14695383#comment-14695383 ] angela commented on OAK-3201: - i can't tell you what would be suited from an OSGi point of view.

[jira] [Commented] (OAK-3201) Use static references in SecurityProviderImpl for composite services

2015-08-12 Thread Francesco Mari (JIRA)
[ https://issues.apache.org/jira/browse/OAK-3201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14693699#comment-14693699 ] Francesco Mari commented on OAK-3201: - I'm not sure if [^OAK-3201-01.patch] solves the p