Solr
  1. Solr
  2. SOLR-4815

Admin-UI - DIH: Let "commit" be checked by default

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Implemented
    • Affects Version/s: None
    • Fix Version/s: 4.4, 6.0
    • Component/s: web gui

      Description

      The new DIH GUI should have "commit" checked by default.

      According to http://wiki.apache.org/solr/DataImportHandler#Commands the REST API has commit=true by default, so it makes sense that the GUI has the same.

      1. SOLR-4815.patch
        0.8 kB
        Stefan Matheis (steffkes)

        Activity

        Hide
        Jan Høydahl added a comment -

        Stefan Matheis (steffkes) could this be a 5-minute fix for 4.4? According to http://wiki.apache.org/solr/DataImportHandler#Commands both "Clean" and "Commit" should default to "true".

        Show
        Jan Høydahl added a comment - Stefan Matheis (steffkes) could this be a 5-minute fix for 4.4? According to http://wiki.apache.org/solr/DataImportHandler#Commands both "Clean" and "Commit" should default to "true".
        Hide
        ASF subversion and git services added a comment -

        Commit 1499252 from steffkes@apache.org
        [ https://svn.apache.org/r1499252 ]

        SOLR-4815: Admin-UI - DIH: Let "commit" be checked by default

        Show
        ASF subversion and git services added a comment - Commit 1499252 from steffkes@apache.org [ https://svn.apache.org/r1499252 ] SOLR-4815 : Admin-UI - DIH: Let "commit" be checked by default
        Hide
        ASF subversion and git services added a comment -

        Commit 1499254 from steffkes@apache.org
        [ https://svn.apache.org/r1499254 ]

        SOLR-4815: Admin-UI - DIH: Let "commit" be checked by default (merge r1499252)

        Show
        ASF subversion and git services added a comment - Commit 1499254 from steffkes@apache.org [ https://svn.apache.org/r1499254 ] SOLR-4815 : Admin-UI - DIH: Let "commit" be checked by default (merge r1499252)
        Hide
        Stefan Matheis (steffkes) added a comment -

        Indeed Jan Høydahl, thanks for hint

        Show
        Stefan Matheis (steffkes) added a comment - Indeed Jan Høydahl , thanks for hint
        Hide
        Steve Rowe added a comment -

        Bulk close resolved 4.4 issues

        Show
        Steve Rowe added a comment - Bulk close resolved 4.4 issues

          People

          • Assignee:
            Stefan Matheis (steffkes)
            Reporter:
            Jan Høydahl
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development