[
https://issues.apache.org/jira/browse/OAK-740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13621850#comment-13621850
]
Tommaso Teofili commented on OAK-740:
-
the wrong part is the not escaped slash, it should
[
https://issues.apache.org/jira/browse/OAK-740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13621850#comment-13621850
]
Tommaso Teofili edited comment on OAK-740 at 4/4/13 6:25 AM:
-
the
[
https://issues.apache.org/jira/browse/OAK-744?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Dürig resolved OAK-744.
---
Resolution: Fixed
Fix Version/s: 0.7
Fixed at revision 1464088
> Invalid base no
[
https://issues.apache.org/jira/browse/OAK-744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13621021#comment-13621021
]
Michael Dürig commented on OAK-744:
---
Revision 1464088 removes the {{baseState}} field entir
Michael Dürig created OAK-744:
-
Summary: Invalid base node states in TreeImpl after
commit/refresh/rebase
Key: OAK-744
URL: https://issues.apache.org/jira/browse/OAK-744
Project: Jackrabbit Oak
[
https://issues.apache.org/jira/browse/OAK-709?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jukka Zitting updated OAK-709:
--
Attachment: SecureNodeState.java
See the attached {{SecureNodeState}} class for a quick draft of how I'd s
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620998#comment-13620998
]
Alex Parvulescu commented on OAK-734:
-
I got it. It's the _CountingNodeVisitor_ which acc
[
https://issues.apache.org/jira/browse/OAK-736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620991#comment-13620991
]
angela commented on OAK-736:
what's the benefit of having two constructors here? apart from tests
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alex Parvulescu updated OAK-734:
Attachment: OAK-734-p2.patch
Attaching part 2, removing the insert/delete sets.
Look at 'Property2Ind
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620983#comment-13620983
]
Alex Parvulescu commented on OAK-734:
-
I took one test that seemed longer and the first c
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620975#comment-13620975
]
Jukka Zitting commented on OAK-734:
---
bq. apparently there's a price for the direct updates
[
https://issues.apache.org/jira/browse/OAK-709?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620970#comment-13620970
]
angela commented on OAK-709:
as stated on the list i like the idea of having the access check on
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620968#comment-13620968
]
Alex Parvulescu commented on OAK-734:
-
pushed initial version of the patch with rev 14640
[
https://issues.apache.org/jira/browse/OAK-743?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620960#comment-13620960
]
Marcel Reutegger commented on OAK-743:
--
We already have the feature flag :)
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620956#comment-13620956
]
Michael Dürig commented on OAK-675:
---
With the current design there is no reliable way to as
[
https://issues.apache.org/jira/browse/OAK-742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620952#comment-13620952
]
Antonio Sanso commented on OAK-742:
---
For what I can see this method is only used from the t
[
https://issues.apache.org/jira/browse/OAK-743?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620949#comment-13620949
]
Michael Dürig commented on OAK-743:
---
+1 for increasing that value. However, let's introduce
Marcel Reutegger created OAK-743:
Summary: Increase default RootImpl.PURGE_LIMIT
Key: OAK-743
URL: https://issues.apache.org/jira/browse/OAK-743
Project: Jackrabbit Oak
Issue Type: Improvemen
[
https://issues.apache.org/jira/browse/OAK-742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620943#comment-13620943
]
Jukka Zitting commented on OAK-742:
---
> to avoid any misusage
What kind of misuse were you
[
https://issues.apache.org/jira/browse/OAK-640?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
angela resolved OAK-640.
Resolution: Fixed
resolving issue.
> Add WorkspaceInitializer
>
>
>
Antonio Sanso created OAK-742:
-
Summary: Oak#createContentSession() should be moved to some Test
class
Key: OAK-742
URL: https://issues.apache.org/jira/browse/OAK-742
Project: Jackrabbit Oak
Iss
Jukka Zitting created OAK-741:
-
Summary: Better toString() methods
Key: OAK-741
URL: https://issues.apache.org/jira/browse/OAK-741
Project: Jackrabbit Oak
Issue Type: Improvement
Compon
[
https://issues.apache.org/jira/browse/OAK-703?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
angela resolved OAK-703.
Resolution: Fixed
Committed revision 1464026.
> Drop SessionImport#checkPermission
>
[
https://issues.apache.org/jira/browse/OAK-706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620929#comment-13620929
]
angela commented on OAK-706:
recently added a workaround by adding a utility method to ImmutableR
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620924#comment-13620924
]
Alex Parvulescu commented on OAK-734:
-
no worries, I've created OAK-740 to track the erro
Alex Parvulescu created OAK-740:
---
Summary: Malformed solr delete query
Key: OAK-740
URL: https://issues.apache.org/jira/browse/OAK-740
Project: Jackrabbit Oak
Issue Type: Bug
Componen
[
https://issues.apache.org/jira/browse/OAK-426?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Dürig resolved OAK-426.
---
Resolution: Won't Fix
Fix Version/s: 0.7
This is superseded by OAK-732 now.
> OA
[
https://issues.apache.org/jira/browse/OAK-732?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Dürig resolved OAK-732.
---
Resolution: Fixed
Fix Version/s: 0.7
Assignee: Michael Dürig
Fixed at revision 1463984
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620902#comment-13620902
]
Tommaso Teofili commented on OAK-734:
-
thanks Alex for digging into it, I think you're ri
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620893#comment-13620893
]
Alex Parvulescu commented on OAK-734:
-
I dug deeper into the solr error. It appears that
[
https://issues.apache.org/jira/browse/OAK-739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620891#comment-13620891
]
Michael Dürig commented on OAK-739:
---
Same problem with {{RestrictionDefinitionImpl}} and {{
[
https://issues.apache.org/jira/browse/OAK-509?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Dürig resolved OAK-509.
---
Resolution: Fixed
Fix Version/s: 0.7
Fixed at revision 1463964.
I added additional test coverage
[
https://issues.apache.org/jira/browse/OAK-739?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Lukas Eder updated OAK-739:
---
Description:
I was digging in the Oak code and I've stumbled upon an assymetric equals()
implementation between
Lukas Eder created OAK-739:
--
Summary: Asymmetric equals() implementation in
AccessControlManagerImpl's nested classes PrincipalACL / NodeACL
Key: OAK-739
URL: https://issues.apache.org/jira/browse/OAK-739
Pr
[
https://issues.apache.org/jira/browse/OAK-737?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
angela resolved OAK-737.
Resolution: Fixed
fixed in Revision: 1463955
> Remove default security provider in Oak.java
> ---
[
https://issues.apache.org/jira/browse/OAK-737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620881#comment-13620881
]
angela commented on OAK-737:
i don't want to change the default but want to get rid of the defaul
[
https://issues.apache.org/jira/browse/OAK-737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620875#comment-13620875
]
Antonio Sanso commented on OAK-737:
---
blindly changing the oak security provider to
privat
[
https://issues.apache.org/jira/browse/OAK-509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620866#comment-13620866
]
Michael Dürig commented on OAK-509:
---
Yes, {{LocalNameMapper.getJcrName}} should not do
{{c
[
https://issues.apache.org/jira/browse/OAK-738?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Dürig resolved OAK-738.
---
Resolution: Fixed
Fix Version/s: 0.7
Fixed at revision 1463931
> Assertion error
[
https://issues.apache.org/jira/browse/OAK-738?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Dürig reassigned OAK-738:
-
Assignee: Michael Dürig
> Assertion error when adding node with expanded name
>
Michael Dürig created OAK-738:
-
Summary: Assertion error when adding node with expanded name
Key: OAK-738
URL: https://issues.apache.org/jira/browse/OAK-738
Project: Jackrabbit Oak
Issue Type: Bu
[
https://issues.apache.org/jira/browse/OAK-737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620800#comment-13620800
]
Antonio Sanso commented on OAK-737:
---
+1
> Remove default security provider
[
https://issues.apache.org/jira/browse/OAK-737?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
angela reassigned OAK-737:
--
Assignee: angela
> Remove default security provider in Oak.java
>
angela created OAK-737:
--
Summary: Remove default security provider in Oak.java
Key: OAK-737
URL: https://issues.apache.org/jira/browse/OAK-737
Project: Jackrabbit Oak
Issue Type: Improvement
C
[
https://issues.apache.org/jira/browse/OAK-736?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
angela reassigned OAK-736:
--
Assignee: angela
> Enhancement to the PermissionProviderImpl constructor
> ---
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620786#comment-13620786
]
Tommaso Teofili commented on OAK-734:
-
this is what I get:
{code}
---
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620781#comment-13620781
]
Alex Parvulescu commented on OAK-734:
-
bq. the changes break the sql2 test
hmm, this is
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620777#comment-13620777
]
Tommaso Teofili commented on OAK-734:
-
it generally looks good to me, but regarding the S
[
https://issues.apache.org/jira/browse/OAK-736?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Antonio Sanso updated OAK-736:
--
Attachment: OAK-736-patch.txt
attaching simple patch
> Enhancement to the PermissionProvi
[
https://issues.apache.org/jira/browse/OAK-736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620770#comment-13620770
]
Antonio Sanso commented on OAK-736:
---
Attaching simple patch
#P oak-core
Index:
src/main/
[
https://issues.apache.org/jira/browse/OAK-736?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Antonio Sanso updated OAK-736:
--
Comment: was deleted
(was: Attaching simple patch
#P oak-core
Index:
src/main/java/org/apache/jackrabbi
Antonio Sanso created OAK-736:
-
Summary: Enhancement to the PermissionProviderImpl constructor
Key: OAK-736
URL: https://issues.apache.org/jira/browse/OAK-736
Project: Jackrabbit Oak
Issue Type:
[
https://issues.apache.org/jira/browse/OAK-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620755#comment-13620755
]
Jukka Zitting commented on OAK-734:
---
+1 Looks good to me.
There's a number of further impr
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jukka Zitting reopened OAK-675:
---
Assignee: (was: Alex Parvulescu)
> Observation generates NPE in an existing EventListener
> --
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jukka Zitting updated OAK-675:
--
Issue Type: Improvement (was: Bug)
Summary: Observation generates NPE in an existing EventListener
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620743#comment-13620743
]
Michael Dürig commented on OAK-675:
---
bq. shouldn't this go into OAK-144, or a dedicated sub
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620741#comment-13620741
]
Alex Parvulescu commented on OAK-675:
-
shouldn't this go into OAK-144, or a dedicated sub
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620737#comment-13620737
]
Marcel Reutegger commented on OAK-675:
--
Returning null actually is a problem for other c
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13620728#comment-13620728
]
Jukka Zitting commented on OAK-675:
---
If this turns out to be a problem for also other clien
[
https://issues.apache.org/jira/browse/OAK-675?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alex Parvulescu resolved OAK-675.
-
Resolution: Not A Problem
Assignee: Alex Parvulescu
marking as 'not a problem'. this was addre
60 matches
Mail list logo