headers
Marcel Reutegger (JIRA | 2 Sep 2008 10:36
Picon
Favicon

Created: (JCR-1728) Observation logs error when a node is moved in place

Observation logs error when a node is moved in place
----------------------------------------------------

                 Key: JCR-1728
                 URL: https://issues.apache.org/jira/browse/JCR-1728
             Project: Jackrabbit
          Issue Type: Bug
          Components: jackrabbit-core
    Affects Versions: 1.4
            Reporter: Marcel Reutegger
            Priority: Minor

An error message is written to the log when the following sequence of operations is executed:

- create node 'parent'
- create node 'child' as a child of 'parent'
- save
- create node 'tmp'
- move 'child' under 'tmp'
- remove 'parent'
- move 'tmp' to former path of 'parent'

The log will say: EventStateCollection: Unable to calculate old path of moved node

This is because the zombie path of 'child' is equal to the new path after the move. The EventStateCollection
detects a new parentId assigned to 'child' and expects a new path that is different from the zombie path.
The above case however shows that there is a use case where the paths are equal and events should be generated.
Permalink | Reply | 3 comments |
headers
Marcel Reutegger (JIRA | 2 Sep 2008 12:02
Picon
Favicon

Resolved: (JCR-1728) Observation logs error when a node is moved in place


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

Marcel Reutegger resolved JCR-1728.
-----------------------------------

       Resolution: Fixed
    Fix Version/s: 1.5

Fixed in revision: 691181

> Observation logs error when a node is moved in place
> ----------------------------------------------------
>
>                 Key: JCR-1728
>                 URL: https://issues.apache.org/jira/browse/JCR-1728
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-core
>    Affects Versions: 1.4
>            Reporter: Marcel Reutegger
>            Priority: Minor
>             Fix For: 1.5
>
>
> An error message is written to the log when the following sequence of operations is executed:
> - create node 'parent'
> - create node 'child' as a child of 'parent'
> - save
(Continue reading)

Permalink | Reply | 0 comments |
headers
Jukka Zitting (JIRA | 2 Sep 2008 17:59
Picon
Favicon

Commented: (JCR-1705) Reduce memory usage of transient nodes


    [
https://issues.apache.org/jira/browse/JCR-1705?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12627720#action_12627720
] 

Jukka Zitting commented on JCR-1705:
------------------------------------

+1 Looks great! I'm getting some impressive memory savings with this patch when importing large numbers of
nodes through the transient space.

> Reduce memory usage of transient nodes
> --------------------------------------
>
>                 Key: JCR-1705
>                 URL: https://issues.apache.org/jira/browse/JCR-1705
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Marcel Reutegger
>            Priority: Minor
>         Attachments: JCR-1705-refactored.patch, JCR-1705-refactored.patch, JCR-1705.patch, JCR-1705.patch
>
>
> When adding lots of transient nodes, most of them don't have child nodes because they are leafs. The
attached patch initializes NodeState.childNodeEntries with an unmodifiable empty ChildNodeEntries
instance and turns it into a modifiable one only when needed.
> Running a test with 100k nodes (10 children per node) the memory consumption for child node entries drops
from 42MB to 12MB with this patch.
(Continue reading)

Permalink | Reply | 0 comments |
headers
Jukka Zitting (JIRA | 2 Sep 2008 18:29
Picon
Favicon

Updated: (JCR-1538) [patch] add toString for NodeImpl and PropertyImpl


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

Jukka Zitting updated JCR-1538:
-------------------------------

    Remaining Estimate: 0h
     Original Estimate: 0h

I made some related but less complex improvements in revisions 691296 and 691305. All items now return
their type ("node" or "property") and (safe) path as the toString() output.

> [patch] add toString for NodeImpl and PropertyImpl
> --------------------------------------------------
>
>                 Key: JCR-1538
>                 URL: https://issues.apache.org/jira/browse/JCR-1538
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-core
>    Affects Versions: core 1.4.2
>            Reporter: Dave Brosius
>            Priority: Trivial
>             Fix For: 1.5
>
>         Attachments: node_and_property_toString.patch
>
>   Original Estimate: 0h
>  Remaining Estimate: 0h
(Continue reading)

Permalink | Reply | 0 comments |
headers
Jukka Zitting (JIRA | 2 Sep 2008 18:35
Picon
Favicon

Updated: (JCR-1522) [PATCH] Java Class file text extractor


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

Jukka Zitting updated JCR-1522:
-------------------------------

    Remaining Estimate: 0h
     Original Estimate: 0h

I'm not too worried about a text extractor dependency, as text extractors are well insulated from the rest
of the system and it's easy for a downstream project to simply exclude the dependency if it causes trouble.

PS. Dave, I assume it's OK if I use this contribution also in Apache Tika?

> [PATCH] Java Class file text extractor
> --------------------------------------
>
>                 Key: JCR-1522
>                 URL: https://issues.apache.org/jira/browse/JCR-1522
>             Project: Jackrabbit
>          Issue Type: New Feature
>          Components: jackrabbit-text-extractors
>    Affects Versions: 1.4.1
>            Reporter: Dave Brosius
>            Priority: Trivial
>             Fix For: 1.4.1
>
>         Attachments: classfile_text_extractor.patch, classfile_text_extractor.patch
>
(Continue reading)

Permalink | Reply | 0 comments |
headers
Jukka Zitting (JIRA | 2 Sep 2008 18:35
Picon
Favicon

Resolved: (JCR-1506) [PATCH] NodePropBundle can only calculate int sizes from getSize()


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

Jukka Zitting resolved JCR-1506.
--------------------------------

    Resolution: Won't Fix
      Assignee: Jukka Zitting

Resolved as Won't Fix.

> [PATCH] NodePropBundle can only calculate int sizes from getSize()
> ------------------------------------------------------------------
>
>                 Key: JCR-1506
>                 URL: https://issues.apache.org/jira/browse/JCR-1506
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Dave Brosius
>            Assignee: Jukka Zitting
>            Priority: Trivial
>         Attachments: use_long_space.patch
>
>
> NodePropBundle.getSize() does integer math, then casts it to a long, thus limiting the range of values to
just those that fit in an integer. This patch fixes this.
(Continue reading)

Permalink | Reply | 0 comments |
headers
Jukka Zitting (JIRA | 2 Sep 2008 18:41
Picon
Favicon

Resolved: (JCR-1445) [PATCH] Use entrySet iterators to avoid map look ups in loops


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

Jukka Zitting resolved JCR-1445.
--------------------------------

    Resolution: Fixed
      Assignee: Jukka Zitting

Patch applied in revision 691313. Thanks!

> [PATCH] Use entrySet iterators to avoid map look ups in loops
> -------------------------------------------------------------
>
>                 Key: JCR-1445
>                 URL: https://issues.apache.org/jira/browse/JCR-1445
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Dave Brosius
>            Assignee: Jukka Zitting
>            Priority: Trivial
>             Fix For: 1.5
>
>         Attachments: use_entry_sets.patch
>
>
> Code uses a keySet iterator in a loop, then does a map look up using the key retrieved from the iterator. 
> Might as well use an entrySet iterator to avoid n map lookups.
(Continue reading)

Permalink | Reply | 0 comments |
headers
Jukka Zitting (JIRA | 2 Sep 2008 18:47
Picon
Favicon

Resolved: (JCR-1404) javadoc writing and generation with mvn


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

Jukka Zitting resolved JCR-1404.
--------------------------------

    Resolution: Fixed
      Assignee: Jukka Zitting

POM improvement applied in revision 691315. Thanks!

> javadoc writing and generation with mvn
> ---------------------------------------
>
>                 Key: JCR-1404
>                 URL: https://issues.apache.org/jira/browse/JCR-1404
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-ocm
>         Environment: Linux sun jdk 1.6
>            Reporter: Alex Lukin
>            Assignee: Jukka Zitting
>            Priority: Trivial
>             Fix For: 1.5
>
>   Original Estimate: 120h
>  Remaining Estimate: 120h
>
> "mvn -source 1.5 javadoc:javadoc" does not work because following lines must be added to pom.xml
(Continue reading)

Permalink | Reply | 0 comments |
headers
Jukka Zitting (JIRA | 2 Sep 2008 19:05
Picon
Favicon

Resolved: (JCR-1317) Add a MBean method to programatically create a new Workspace.


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

Jukka Zitting resolved JCR-1317.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 1.5
         Assignee: Jukka Zitting

Patch applied in revision 691319 and polished in revision 691321. Thanks!

> Add a MBean method to programatically create a new Workspace.
> -------------------------------------------------------------
>
>                 Key: JCR-1317
>                 URL: https://issues.apache.org/jira/browse/JCR-1317
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-jcr-rmi
>    Affects Versions: 1.3.3
>            Reporter: Manuel Duran
>            Assignee: Jukka Zitting
>            Priority: Trivial
>             Fix For: 1.5
>
>         Attachments: createWorkdpace.patch
>
>
(Continue reading)

Permalink | Reply | 0 comments |
headers
Dave Brosius (JIRA | 2 Sep 2008 19:19
Picon
Favicon

Commented: (JCR-1456) Database connection pooling


    [
https://issues.apache.org/jira/browse/JCR-1456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12627752#action_12627752
] 

Dave Brosius commented on JCR-1456:
-----------------------------------

I like the idea of this patch, but i think the ConnectionProperties is too specific. It should just be

private String url;
private String driver;
private Properties connectionProperties.

for instance, i would like to add

properties.put("oracle.net.ssl_cipher_suites", "(SSL_DH_anon_WITH_3DES_EDE_CBC_SHA,
SSL_DH_anon_WITH_RC4_128_MD5, SSL_DH_anon_WITH_DES_CBC_SHA)";

so that i can connect to the jackrabbit database over SSL.

And of course a similar implication for the repository.xml, to include arbitrary connection properties.

> Database connection pooling
> ---------------------------
>
>                 Key: JCR-1456
>                 URL: https://issues.apache.org/jira/browse/JCR-1456
>             Project: Jackrabbit
>          Issue Type: Improvement
(Continue reading)

Permalink | Reply | 0 comments |

Gmane