gerlowskija opened a new pull request, #2079:
URL: https://github.com/apache/solr/pull/2079

   https://issues.apache.org/jira/browse/SOLR-16835
   
   
   # Description
   
   Now that Solr uses its OAS to generate client bindings in multiple languages 
(Java and JavaScript, with Python coming soon in #1681 ), users of these 
clients may wish to use them to run primitive searches.
   
   Normally, this would require converting the `/select` endpoint to JAX-RS so 
that its inputs and outputs can be described comprehensively in our spec.  
However, doing this for `/select` will take some time due to the complexity and 
large degree of configurability the endpoint offers. 
   
   # Solution
   
   This commit works around this limitation by creating an 'api' module 
interface approximating the `/select` endpoint.  This allows an incomplete 
/select endpoint to appear in our OAS (and the generated clients, by extension) 
until the API can be converted to JAX-RS in earnest.  This will give 
generated-client users access to at least some query functionality in this 
interim.
   
   The `/select` query parameters supported in this commit were chosen mostly 
arbitrarily.  They may be added to freely as generated-client users run into 
particular needs (e.g. for setting 'faceting' parameters).
   
   # Tests
   
   Manual testing of the generated 'QueryingApi' request and response classes.  
Inspection of the generated Javascript (and Python - when combined with #1681) 
client code.  All existing tests continue to pass.
   
   # Checklist
   
   Please review the following and check all that apply:
   
   - [x] I have reviewed the guidelines for [How to 
Contribute](https://wiki.apache.org/solr/HowToContribute) and my code conforms 
to the standards described there to the best of my ability.
   - [x] I have created a Jira issue and added the issue ID to my pull request 
title.
   - [x] I have given Solr maintainers 
[access](https://help.github.com/en/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork)
 to contribute to my PR branch. (optional but recommended)
   - [x] I have developed this patch against the `main` branch.
   - [ ] I have run `./gradlew check`.
   - [ ] I have added tests for my changes.
   - [ ] I have added documentation for the [Reference 
Guide](https://github.com/apache/solr/tree/main/solr/solr-ref-guide)
   


-- 
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

Reply via email to