[ 
https://issues.apache.org/jira/browse/SOLR-11722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16321275#comment-16321275
 ] 

Gus Heck commented on SOLR-11722:
---------------------------------

Attached a patch, I believe I addressed the above comments I did switch to 
using LocalSolrRequest, but that did not alleviate the need for the v2-v1 and 
parseCollectionCreationProps changes because they get utilized in the _OP enum 
to masage the incomming parameter before requests can be made. rather than in 
the Cmd class where delegation to the Command infrastructure via 
LocalSolrRequest happens. 

Names should be mostly in sync with your changes in SOLR-11653, and I extracted 
your collection creation code into createCollectionAndWait() and reused that 
(which is how I picked up LocalSolrRequest :) ).

What's not in this patch is documentation, but I want to get this out where you 
and others can look at it. Working on docs now, which could be added as an 
independent patch if you want to commit this one.


> API to create a Time Routed Alias and first collection
> ------------------------------------------------------
>
>                 Key: SOLR-11722
>                 URL: https://issues.apache.org/jira/browse/SOLR-11722
>             Project: Solr
>          Issue Type: Sub-task
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: SolrCloud
>            Reporter: David Smiley
>         Attachments: SOLR-11722.patch, SOLR-11722.patch
>
>
> This issue is about creating a single API command to create a "Time Routed 
> Alias" along with its first collection.  Need to decide what endpoint URL it 
> is and parameters.
> Perhaps in v2 it'd be {{/api/collections?command=create-routed-alias}} or 
> alternatively piggy-back off of command=create-alias but we add more options, 
> perhaps with a prefix like "router"?
> Inputs:
> * alias name
> * misc collection creation metadata (e.g. config, numShards, ...) perhaps in 
> this context with a prefix like "collection."
> * metadata for TimeRoutedAliasUpdateProcessor, currently: router.field
> * date specifier for first collection; can include "date math".
> We'll certainly add more options as future features unfold.
> I believe the collection needs to be created first (referring to the alias 
> name via a core property), and then the alias pointing to it which demands 
> collections exist first.  When figuring the collection name, you'll need to 
> reference the format in TimeRoutedAliasUpdateProcessor.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to