You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Alex Parvulescu (JIRA)" <ji...@apache.org> on 2011/04/13 12:44:05 UTC
[jira] [Created] (JCR-2944) Remove unnecessary NodeImpl references
from LuceneQueryFactory
Remove unnecessary NodeImpl references from LuceneQueryFactory
--------------------------------------------------------------
Key: JCR-2944
URL: https://issues.apache.org/jira/browse/JCR-2944
Project: Jackrabbit Content Repository
Issue Type: Improvement
Components: query
Reporter: Alex Parvulescu
Priority: Trivial
LuceneQueryFactory casts to NodeImpl just to get the node id.
This info is available via the api as well, so the cast seems unnecessary.
I'll attach a patch for this tiny issue.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (JCR-2944) Remove unnecessary NodeImpl references
from LuceneQueryFactory
Posted by "Alex Parvulescu (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/JCR-2944?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alex Parvulescu updated JCR-2944:
---------------------------------
Attachment: JCR-2944-v2.patch
reattached the patch. I initially forgot to clean the imports too
> Remove unnecessary NodeImpl references from LuceneQueryFactory
> --------------------------------------------------------------
>
> Key: JCR-2944
> URL: https://issues.apache.org/jira/browse/JCR-2944
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: query
> Reporter: Alex Parvulescu
> Priority: Trivial
> Attachments: JCR-2944-v2.patch, JCR-2944.patch
>
>
> LuceneQueryFactory casts to NodeImpl just to get the node id.
> This info is available via the api as well, so the cast seems unnecessary.
> I'll attach a patch for this tiny issue.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (JCR-2944) Remove unnecessary NodeImpl references
from LuceneQueryFactory
Posted by "Alex Parvulescu (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/JCR-2944?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alex Parvulescu updated JCR-2944:
---------------------------------
Attachment: JCR-2944.patch
attached patch to remove unnecessary casts
> Remove unnecessary NodeImpl references from LuceneQueryFactory
> --------------------------------------------------------------
>
> Key: JCR-2944
> URL: https://issues.apache.org/jira/browse/JCR-2944
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: query
> Reporter: Alex Parvulescu
> Priority: Trivial
> Attachments: JCR-2944.patch
>
>
> LuceneQueryFactory casts to NodeImpl just to get the node id.
> This info is available via the api as well, so the cast seems unnecessary.
> I'll attach a patch for this tiny issue.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Resolved] (JCR-2944) Remove unnecessary NodeImpl references
from LuceneQueryFactory
Posted by "Jukka Zitting (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/JCR-2944?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jukka Zitting resolved JCR-2944.
--------------------------------
Resolution: Fixed
Fix Version/s: 2.3.0
Assignee: Jukka Zitting
Thanks! Patch committed in revision 1092736.
> Remove unnecessary NodeImpl references from LuceneQueryFactory
> --------------------------------------------------------------
>
> Key: JCR-2944
> URL: https://issues.apache.org/jira/browse/JCR-2944
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: query
> Reporter: Alex Parvulescu
> Assignee: Jukka Zitting
> Priority: Trivial
> Fix For: 2.3.0
>
> Attachments: JCR-2944-v2.patch, JCR-2944.patch
>
>
> LuceneQueryFactory casts to NodeImpl just to get the node id.
> This info is available via the api as well, so the cast seems unnecessary.
> I'll attach a patch for this tiny issue.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira