Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-15781 Make v2 APIs more REST-ful and migrate to JAX-RS
  3. SOLR-16390

Cosmetic improvements and migration to JAX-RS (v2 cluster and clusterprop APIs)

    XMLWordPrintableJSON

Details

    Description

      As mentioned on SOLR-15781, the v2 API currently has an experimental designation, and the community has expressed an interest in using this period to update our v2 endpoints to be more REST-ful and consistent. The current plan is to follow the specific changes laid out in this spreadsheet, though of course nothing there is set in stone and there are still warts to be worked out.

      This ticket plans to tackle making the changes required for Solr's "Cluster" and "cluster-prop" APIs. These APIs are described in detail in the spreadsheet linked above, but are summarized in the table below for convenience and easier tracking.

      While we're touching the code for these endpoints, we should also convert them to JAX-RS framework definitions. (This was initially tracked as a separate effort - see SOLR-16370 - but the edit that were required ended up overlapping so significantly with the "cosmetic" improvements here that in practice it almost always makes sense to do the two together.)

      Cosmetic Changes and JAX-RS Conversion

      API Name Original Form Desired Form Status Volunteer
      Upsert ClusterProp POST /api/cluster {"set-property": {...}} PUT /api/cluster/properties/propName {"value": "propVal"} Open N/A
      Upsert ClusterProp (Potentially Nested) POST /api/cluster {"set-obj-property": {...}} PUT /api/cluster/properties {...} Open N/A
      Delete ClusterProp POST /api/cluster {"set-property": {"name": "foo", "value": ""}} DELETE /api/cluster/properties/propName Open N/A
      Delete Single Async Status DELETE /api/cluster/command-status/123 DELETE /api/cluster/commands/123 Open N/A
      Delete All Async Status DELETE /api/cluster/command-status DELETE /api/cluster/commands Open N/A
      Get Single Async Status GET /api/cluster/command-status/123 GET /api/cluster/commands/123 Open N/A
      Create rate-limiter POST /api/cluster/ {"set-ratelimiter": {...}} POST /api/cluster/ratelimiter {...} Open N/A

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              gerlowskija Jason Gerlowski
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 4h 10m
                  4h 10m