Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-4204

be test failure: KuduScanNodeTest: TestLimitsAreEnforced: row mismatch

    Details

      Description

      One failure so far.

          <testcase name="TestLimitsAreEnforced" status="run" time="0.438" classname="KuduScanNodeTest">
            <failure message="Value of: row&#x0A;  Actual: &quot;[(0 0 hello_0)]&quot;&#x0A;Expected: rows[i]&#x0A;Which is: &quot;[(5 10 hello_5)]&quot;" type=""><![CDATA[/data/jenkins/workspace/impala-umbrella-build-and-test/repos/Impala/be/src/exec/kudu-scan-node-test.cc:152
      Value of: row
        Actual: "[(0 0 hello_0)]"
      Expected: rows[i]
      Which is: "[(5 10 hello_5)]"]]></failure>
          </testcase>
      

      http://sandbox.jenkins.cloudera.com/view/Impala/view/Evergreen-asf-master/job/impala-asf-master-exhaustive-release/

        Activity

        Hide
        mikesbrown Michael Brown added a comment -

        I know Matthew Jacobs is at a conference, but so far this has happened once. We can reconsider assignee if this causes more pain.

        Show
        mikesbrown Michael Brown added a comment - I know Matthew Jacobs is at a conference, but so far this has happened once. We can reconsider assignee if this causes more pain.
        Show
        hsheinblatt_impala_e511 Harrison Sheinblatt added a comment - This same failure occurred in data load on 9/29: http://sandbox.jenkins.sf.cloudera.com/view/Impala/view/Evergreen-asf-master/job/impala-asf-master-core-data-load/86/testReport/junit/(root)/KuduScanNodeTest/TestLimitsAreEnforced/ Marking as build blocker
        Hide
        mjacobs Matthew Jacobs added a comment -

        Turns out this test is flaky. If the scanner threads (there are 2 in this case) return a batch from the latter range (starting at id 5) first, the test fails. Marcel had previously suggested removing these tests since they are hard to maintain and we don't have BE unit tests for other scan nodes. Now that we have functional EE tests in place for Kudu, I think we can remove them.

        Show
        mjacobs Matthew Jacobs added a comment - Turns out this test is flaky. If the scanner threads (there are 2 in this case) return a batch from the latter range (starting at id 5) first, the test fails. Marcel had previously suggested removing these tests since they are hard to maintain and we don't have BE unit tests for other scan nodes. Now that we have functional EE tests in place for Kudu, I think we can remove them.
        Hide
        mjacobs Matthew Jacobs added a comment -

        commit 485022a89921592fdaa12deffe27923832103d84
        Author: Matthew Jacobs <mj@cloudera.com>
        Date: Mon Oct 3 14:01:27 2016 -0700

        IMPALA-4204: Remove KuduScanNodeTest

        In IMPALA-4204, the limit test could fail depending on the
        timing of the scanner threads.

        Now that we have functional tests for Kudu (IMPALA-3718), we
        can remove the BE KuduScanNode tests which have been error
        prone and haven't been any new bugs. Also, we don't have BE
        tests for any other scan nodes.

        Change-Id: I17c050f461c5cf0649857b82d747d87df0a641b8
        Reviewed-on: http://gerrit.cloudera.org:8080/4609
        Reviewed-by: Alex Behm <alex.behm@cloudera.com>
        Reviewed-by: Marcel Kornacker <marcel@cloudera.com>
        Tested-by: Internal Jenkins

        Show
        mjacobs Matthew Jacobs added a comment - commit 485022a89921592fdaa12deffe27923832103d84 Author: Matthew Jacobs <mj@cloudera.com> Date: Mon Oct 3 14:01:27 2016 -0700 IMPALA-4204 : Remove KuduScanNodeTest In IMPALA-4204 , the limit test could fail depending on the timing of the scanner threads. Now that we have functional tests for Kudu ( IMPALA-3718 ), we can remove the BE KuduScanNode tests which have been error prone and haven't been any new bugs. Also, we don't have BE tests for any other scan nodes. Change-Id: I17c050f461c5cf0649857b82d747d87df0a641b8 Reviewed-on: http://gerrit.cloudera.org:8080/4609 Reviewed-by: Alex Behm <alex.behm@cloudera.com> Reviewed-by: Marcel Kornacker <marcel@cloudera.com> Tested-by: Internal Jenkins

          People

          • Assignee:
            mjacobs Matthew Jacobs
            Reporter:
            mikesbrown Michael Brown
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development