Solr
  1. Solr
  2. SOLR-5421

Fix processAdd method in DistributedUpdateProcessor

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.5, 4.5.1
    • Fix Version/s: 4.7, Trunk
    • Component/s: SolrCloud
    • Labels:
      None

      Description

      The processAdd method in DistributedUpdateProcessor sets "distrib.from" in all cases. That shouldn't be the case.

      It should only be set in case the it's a leader/sub-shard leader.

      1. SOLR-5421.patch
        0.6 kB
        Anshum Gupta

        Activity

        Anshum Gupta created issue -
        Anshum Gupta made changes -
        Field Original Value New Value
        Affects Version/s 4.5 [ 12324743 ]
        Affects Version/s 4.5.1 [ 12325264 ]
        Anshum Gupta made changes -
        Attachment SOLR-5421.patch [ 12612108 ]
        Shalin Shekhar Mangar made changes -
        Assignee Shalin Shekhar Mangar [ shalinmangar ]
        Hide
        ASF subversion and git services added a comment -

        Commit 1542177 from shalin@apache.org in branch 'dev/trunk'
        [ https://svn.apache.org/r1542177 ]

        SOLR-5421: Remove double set of distrib.from param in processAdd method of DistributedUpdateProcessor

        Show
        ASF subversion and git services added a comment - Commit 1542177 from shalin@apache.org in branch 'dev/trunk' [ https://svn.apache.org/r1542177 ] SOLR-5421 : Remove double set of distrib.from param in processAdd method of DistributedUpdateProcessor
        Hide
        ASF subversion and git services added a comment -

        Commit 1542178 from shalin@apache.org in branch 'dev/branches/branch_4x'
        [ https://svn.apache.org/r1542178 ]

        SOLR-5421: Remove double set of distrib.from param in processAdd method of DistributedUpdateProcessor

        Show
        ASF subversion and git services added a comment - Commit 1542178 from shalin@apache.org in branch 'dev/branches/branch_4x' [ https://svn.apache.org/r1542178 ] SOLR-5421 : Remove double set of distrib.from param in processAdd method of DistributedUpdateProcessor
        Hide
        Shalin Shekhar Mangar added a comment -

        Thanks Anshum!

        Show
        Shalin Shekhar Mangar added a comment - Thanks Anshum!
        Shalin Shekhar Mangar made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 5.0 [ 12321664 ]
        Fix Version/s 4.7 [ 12325573 ]
        Resolution Fixed [ 1 ]
        David Smiley made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        10d 1h 15m 1 Shalin Shekhar Mangar 15/Nov/13 05:46
        Resolved Resolved Closed Closed
        121d 7h 17m 1 David Smiley 16/Mar/14 13:04

          People

          • Assignee:
            Shalin Shekhar Mangar
            Reporter:
            Anshum Gupta
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development