You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Igor Fedorenko (JIRA)" <ji...@codehaus.org> on 2012/11/05 22:49:13 UTC

[jira] (MINDEXER-65) Provide a way to scan directly to provided IndexingContext

Igor Fedorenko created MINDEXER-65:
--------------------------------------

             Summary: Provide a way to scan directly to provided IndexingContext
                 Key: MINDEXER-65
                 URL: https://jira.codehaus.org/browse/MINDEXER-65
             Project: Maven Indexer
          Issue Type: Improvement
    Affects Versions: 5.0.0
            Reporter: Igor Fedorenko


Currently, DefaultNexusIndexer#scan(...) always perform scan to a temporary indexing context first, then replace the target context with temporary. I vaguely remember this was necessary to allow index queries while scan was running in a background thread. With recent rework of indexer locking, the temporary indexing context is no longer needed and in fact it is harmful when host application requires tight control over IndexingContext status.

Maven indexer need to provide a way to request scan directly to the target indexing context. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] (MINDEXER-65) Provide a way to scan directly to provided IndexingContext

Posted by "Tamás Cservenák (JIRA)" <ji...@codehaus.org>.
     [ https://jira.codehaus.org/browse/MINDEXER-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tamás Cservenák updated MINDEXER-65:
------------------------------------

    Fix Version/s: 5.0.1
         Assignee: Tamás Cservenák
    
> Provide a way to scan directly to provided IndexingContext
> ----------------------------------------------------------
>
>                 Key: MINDEXER-65
>                 URL: https://jira.codehaus.org/browse/MINDEXER-65
>             Project: Maven Indexer
>          Issue Type: Improvement
>    Affects Versions: 5.0.0
>            Reporter: Igor Fedorenko
>            Assignee: Tamás Cservenák
>             Fix For: 5.0.1
>
>         Attachments: 0001-MINDEXER-65-made-DefaultScannerListener-directly-usa.patch
>
>
> Currently, DefaultNexusIndexer#scan(...) always perform scan to a temporary indexing context first, then replace the target context with temporary. I vaguely remember this was necessary to allow index queries while scan was running in a background thread. With recent rework of indexer locking, the temporary indexing context is no longer needed and in fact it is harmful when host application requires tight control over IndexingContext status.
> Maven indexer need to provide a way to request scan directly to the target indexing context. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

[jira] (MINDEXER-65) Provide a way to scan directly to provided IndexingContext

Posted by "Tamás Cservenák (JIRA)" <ji...@codehaus.org>.
    [ https://jira.codehaus.org/browse/MINDEXER-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=313708#comment-313708 ] 

Tamás Cservenák commented on MINDEXER-65:
-----------------------------------------

Fixed in
http://svn.apache.org/viewvc?view=revision&revision=1410252

Curious, why not modify the org.apache.maven.index.DefaultNexusIndexer#scan too? There is really no need to use tmpContext in there anymore...
                
> Provide a way to scan directly to provided IndexingContext
> ----------------------------------------------------------
>
>                 Key: MINDEXER-65
>                 URL: https://jira.codehaus.org/browse/MINDEXER-65
>             Project: Maven Indexer
>          Issue Type: Improvement
>    Affects Versions: 5.0.0
>            Reporter: Igor Fedorenko
>            Assignee: Tamás Cservenák
>             Fix For: 5.1.0
>
>         Attachments: 0001-MINDEXER-65-made-DefaultScannerListener-directly-usa.patch
>
>
> Currently, DefaultNexusIndexer#scan(...) always perform scan to a temporary indexing context first, then replace the target context with temporary. I vaguely remember this was necessary to allow index queries while scan was running in a background thread. With recent rework of indexer locking, the temporary indexing context is no longer needed and in fact it is harmful when host application requires tight control over IndexingContext status.
> Maven indexer need to provide a way to request scan directly to the target indexing context. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

[jira] (MINDEXER-65) Provide a way to scan directly to provided IndexingContext

Posted by "Igor Fedorenko (JIRA)" <ji...@codehaus.org>.
     [ https://jira.codehaus.org/browse/MINDEXER-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Igor Fedorenko updated MINDEXER-65:
-----------------------------------

    Attachment: 0001-MINDEXER-65-made-DefaultScannerListener-directly-usa.patch

proposed fix
                
> Provide a way to scan directly to provided IndexingContext
> ----------------------------------------------------------
>
>                 Key: MINDEXER-65
>                 URL: https://jira.codehaus.org/browse/MINDEXER-65
>             Project: Maven Indexer
>          Issue Type: Improvement
>    Affects Versions: 5.0.0
>            Reporter: Igor Fedorenko
>         Attachments: 0001-MINDEXER-65-made-DefaultScannerListener-directly-usa.patch
>
>
> Currently, DefaultNexusIndexer#scan(...) always perform scan to a temporary indexing context first, then replace the target context with temporary. I vaguely remember this was necessary to allow index queries while scan was running in a background thread. With recent rework of indexer locking, the temporary indexing context is no longer needed and in fact it is harmful when host application requires tight control over IndexingContext status.
> Maven indexer need to provide a way to request scan directly to the target indexing context. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] (MINDEXER-65) Provide a way to scan directly to provided IndexingContext

Posted by "Igor Fedorenko (JIRA)" <ji...@codehaus.org>.
    [ https://jira.codehaus.org/browse/MINDEXER-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=313738#comment-313738 ] 

Igor Fedorenko commented on MINDEXER-65:
----------------------------------------

I am not sure DefaultNexusIndexer#scan is useful at all. I would just deprecated it and suggest direct use of scanner instead.
                
> Provide a way to scan directly to provided IndexingContext
> ----------------------------------------------------------
>
>                 Key: MINDEXER-65
>                 URL: https://jira.codehaus.org/browse/MINDEXER-65
>             Project: Maven Indexer
>          Issue Type: Improvement
>    Affects Versions: 5.0.0
>            Reporter: Igor Fedorenko
>            Assignee: Tamás Cservenák
>             Fix For: 5.1.0
>
>         Attachments: 0001-MINDEXER-65-made-DefaultScannerListener-directly-usa.patch
>
>
> Currently, DefaultNexusIndexer#scan(...) always perform scan to a temporary indexing context first, then replace the target context with temporary. I vaguely remember this was necessary to allow index queries while scan was running in a background thread. With recent rework of indexer locking, the temporary indexing context is no longer needed and in fact it is harmful when host application requires tight control over IndexingContext status.
> Maven indexer need to provide a way to request scan directly to the target indexing context. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

[jira] (MINDEXER-65) Provide a way to scan directly to provided IndexingContext

Posted by "Tamás Cservenák (JIRA)" <ji...@codehaus.org>.
     [ https://jira.codehaus.org/browse/MINDEXER-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tamás Cservenák closed MINDEXER-65.
-----------------------------------

    Resolution: Fixed
    
> Provide a way to scan directly to provided IndexingContext
> ----------------------------------------------------------
>
>                 Key: MINDEXER-65
>                 URL: https://jira.codehaus.org/browse/MINDEXER-65
>             Project: Maven Indexer
>          Issue Type: Improvement
>    Affects Versions: 5.0.0
>            Reporter: Igor Fedorenko
>            Assignee: Tamás Cservenák
>             Fix For: 5.1.0
>
>         Attachments: 0001-MINDEXER-65-made-DefaultScannerListener-directly-usa.patch
>
>
> Currently, DefaultNexusIndexer#scan(...) always perform scan to a temporary indexing context first, then replace the target context with temporary. I vaguely remember this was necessary to allow index queries while scan was running in a background thread. With recent rework of indexer locking, the temporary indexing context is no longer needed and in fact it is harmful when host application requires tight control over IndexingContext status.
> Maven indexer need to provide a way to request scan directly to the target indexing context. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira