dsmiley commented on PR #2809: URL: https://github.com/apache/solr/pull/2809#issuecomment-2508199409
> I agree that it takes an extra step to sopurce control this as you'll need to source control it and then push it to the model store (like it happens with Learning To Rank). I'm not sure it's a blocket though, I need to be cautious about the time I have left to work on this. Can't this be a future improvement? Contributed by others? Ok. I am curious why you did it this way though? I know you saw LTR doing it but it adds significant scope/complexity to this PR that otherwise wouldn't be there; it would have been merged by now and with less code and less of your limited time. I view that as the MVP and model CRUD as a bonus -- come later; more to debate. Put differently, do you see runtime manipulation as an important feature/approach for its viability vs editing solrconfig.xml and sending a RELOAD command? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
