Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      A small improvement to the JSON update format would be to allow for simpler delete-by-id

      {"delete":"myid"}
      {"delete":["id1","id2"]}
      
      1. SOLR-3508_versions.patch
        5 kB
        Yonik Seeley
      2. SOLR-3508.patch
        6 kB
        Yonik Seeley

        Activity

        Hide
        Yonik Seeley added a comment -

        Here's a patch implementing the syntax in the description.

        Show
        Yonik Seeley added a comment - Here's a patch implementing the syntax in the description.
        Hide
        Jack Krupansky added a comment -

        To be clear, the intent is that the value of a "delete" can be an array which contains a list of zero or more id's to be deleted. At first I thought it was a range (start and end), but in fact it is a list of specific id's.

        I presume that somebody will update the doc as well.

        Show
        Jack Krupansky added a comment - To be clear, the intent is that the value of a "delete" can be an array which contains a list of zero or more id's to be deleted. At first I thought it was a range (start and end), but in fact it is a list of specific id's. I presume that somebody will update the doc as well.
        Hide
        Yonik Seeley added a comment -

        While I'm in here mucking with the delete syntax, we should probably have a way to specify the version of deletes in the body of the update request as well (as opposed to a param on the URL). It should prob just sit right beside the current "commitWithin".

        Show
        Yonik Seeley added a comment - While I'm in here mucking with the delete syntax, we should probably have a way to specify the version of deletes in the body of the update request as well (as opposed to a param on the URL). It should prob just sit right beside the current "commitWithin".
        Hide
        Yonik Seeley added a comment -

        Here's a patch that allows the specification of version along with each delete.

        Show
        Yonik Seeley added a comment - Here's a patch that allows the specification of version along with each delete.

          People

          • Assignee:
            Yonik Seeley
            Reporter:
            Yonik Seeley
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development