[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12713024#action_12713024
]
Mark Miller commented on SOLR-243:
--
Yup I know - in the case that that goes in first, its easier to resolve (I had
already put it here). And if it does not, I'll remove it before committing.
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
> Environment: Solr core
>Reporter: John Wang
>Assignee: Mark Miller
> Fix For: 1.5
>
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, SOLR-243.patch,
> SOLR-243.patch, SOLR-243.patch, SOLR-243.patch, SOLR-243.patch,
> SOLR-243.patch, SOLR-243.patch, SOLR-243.patch, SOLR-243.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12713022#action_12713022
]
Shalin Shekhar Mangar commented on SOLR-243:
Mark, this patch still has the code for SOLR-1184.
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
> Environment: Solr core
>Reporter: John Wang
>Assignee: Mark Miller
> Fix For: 1.5
>
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, SOLR-243.patch,
> SOLR-243.patch, SOLR-243.patch, SOLR-243.patch, SOLR-243.patch,
> SOLR-243.patch, SOLR-243.patch, SOLR-243.patch, SOLR-243.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12670666#action_12670666
]
Shalin Shekhar Mangar commented on SOLR-243:
Mark, are you still interested in this issue? If not, should we mark this to
1.5?
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
> Environment: Solr core
>Reporter: John Wang
>Assignee: Hoss Man
> Fix For: 1.4
>
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, SOLR-243.patch,
> SOLR-243.patch, SOLR-243.patch, SOLR-243.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12605447#action_12605447
]
John Wang commented on SOLR-243:
Sorry, I didn't see Hoss's earlier comments.
Thanks
-John
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
> Environment: Solr core
>Reporter: John Wang
>Assignee: Hoss Man
> Fix For: 1.3
>
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12605424#action_12605424
]
Hoss Man commented on SOLR-243:
---
bq. Hoss has marked the issue for 1.3, so it will be in the release.
for the record, i marked it as 1.3 because itwould be nice to see this in 1.3
... but as i said in my 2008-03-13 comment: we need unit tests and example
configuration before i'm willing to commit.
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
> Environment: Solr core
>Reporter: John Wang
>Assignee: Hoss Man
> Fix For: 1.3
>
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12602860#action_12602860
]
Mike Klaas commented on SOLR-243:
-
Hi John,
Hoss has marked the issue for 1.3, so it will be in the release.
-Mike
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
> Environment: Solr core
>Reporter: John Wang
>Assignee: Hoss Man
> Fix For: 1.3
>
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12602844#action_12602844
]
John Wang commented on SOLR-243:
Not sure what do you mean?It was supposed to be a simple addition.
-John
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
> Environment: Solr core
>Reporter: John Wang
>Assignee: Hoss Man
> Fix For: 1.3
>
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12578553#action_12578553
]
Hoss Man commented on SOLR-243:
---
John: first off, i want to apologize for the *extremely* belated patch review
... I know i told you i'd try to look at this months ago, but ... yeah ... life
tends to get in the way of patches.
on the whole, things seem pretty good -- although you were still using an
interface instead of an abstract class. Having the main API for a type of
plugin be an abstract class is an important mechanism to help future proof
ourselves against possible additions we want to make the the plugin API in the
future; if it's an interface we can't really change it once it's released
(because we might break people Impls); if it's an abstract class, we can always
provide a default impl for people.
the biggest concern i have about this patch at the moment is that there are no
tests, and no example configs, so it's hard to be sure it's even working at all
:)
here's what i see as the current todo list for this issue...
1) there are some legacy SolrIndexSearcher constructors that need to delegate
to the SolrCore to get indexReaderFactory ... perhaps we should have a helper
method that decides which SolrIndexConfig to use based on the "name" ?
2) we need tests showing custom IndexReaderFactory getting used (even if it's
just a mock IndexReaderFactory thatsets a boolean to show it's being used) ...
this will also serve as a test that config syntax works.
3) need commented out example of using a custom indexReaderFactory in
example/solr/conf/solrconfig.xml
4) sanity check this against SOLR-465, make sure we aren't painting ourselves
into a corner.
5) we should make IndexReaderFactory use the AbstractPluginLoader stuff and
remove the guts of SolrIndexConfig.loadIndexReaderFactory. It looks like we'll
need to add a "single item" version of load to the AbstractPluginLoader to make
that work well.
#5 is something that can be done after this is committed, but 1-4 are pretty
important. If you can update the patch with some configs/tests i'll think
about the legacy SolrIndexSearcher constructors and try to figure out a good
solution for them
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
>Affects Versions: 1.3
> Environment: Solr core
>Reporter: John Wang
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch, indexReaderFactory.patch, indexReaderFactory.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (SOLR-243) Create a hook to allow custom code to create custom IndexReaders
[
https://issues.apache.org/jira/browse/SOLR-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12576693#action_12576693
]
John Wang commented on SOLR-243:
What is the plan for this patch?
We have a couple of votes for it.
Thanks
-John
> Create a hook to allow custom code to create custom IndexReaders
>
>
> Key: SOLR-243
> URL: https://issues.apache.org/jira/browse/SOLR-243
> Project: Solr
> Issue Type: Improvement
> Components: search
>Affects Versions: 1.3
> Environment: Solr core
>Reporter: John Wang
> Attachments: indexReaderFactory.patch, indexReaderFactory.patch,
> indexReaderFactory.patch
>
>
> I have a customized IndexReader and I want to write a Solr plugin to use my
> derived IndexReader implementation. Currently IndexReader instantiation is
> hard coded to be:
> IndexReader.open(path)
> It would be really useful if this is done thru a plugable factory that can be
> configured, e.g. IndexReaderFactory
> interface IndexReaderFactory{
> IndexReader newReader(String name,String path);
> }
> the default implementation would just return: IndexReader.open(path)
> And in the newSearcher and getSearcher methods in SolrCore class can call the
> current factory implementation to get the IndexReader instance and then build
> the SolrIndexSearcher by passing in the reader.
> It would be really nice to add this improvement soon (This seems to be a
> trivial addition) as our project really depends on this.
> Thanks
> -John
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
