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

test_sort.py::test_analytic_order_by_random fails with assert

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: Impala 2.9.0
    • Fix Version/s: Impala 2.9.0
    • Component/s: Frontend
    • Labels:
    • Epic Color:
      ghx-label-9

      Description

      test_analytic_order_by_random hit a failure on local filesystem build:

      query_test/test_sort.py:197: in test_analytic_order_by_random
      assert results == analytic_results
      E assert [['2', '5', '...1', '0', ...]] == [['4', '3', '7...1', '0', ...]]
      E At index 0 diff: ['2', '5', '7', '3', '1', '0', '6', '4'] != ['4', '3', '7', '5', '1', '0', '6', '2']

      This test was introduced in IMPALA-4731/IMPALA-397/IMPALA-4728, which merged recently.

        Activity

        Hide
        twmarshall Thomas Tauber-Marshall added a comment -

        commit 698f4a34c612985df9ffcf9a9a666ee6c38c5679
        Author: Thomas Tauber-Marshall <tmarshall@cloudera.com>
        Date: Mon May 1 16:13:40 2017 -0700

        IMPALA-5262: test_analytic_order_by_random fails with assert

        This was a poorly written test that relies on assumptions about
        the behavior of 'rand' and the order that rows get processed in
        a table that Impala doesn't actually guarantee.

        The new version is still sensitive to the precise behavior of
        'rand()', but shouldn't be flaky unless that behavior is changed.

        Change-Id: If1ba8154c2b6a8d508916d85391b95885ef915a9
        Reviewed-on: http://gerrit.cloudera.org:8080/6775
        Reviewed-by: Alex Behm <alex.behm@cloudera.com>
        Tested-by: Impala Public Jenkins

        Show
        twmarshall Thomas Tauber-Marshall added a comment - commit 698f4a34c612985df9ffcf9a9a666ee6c38c5679 Author: Thomas Tauber-Marshall <tmarshall@cloudera.com> Date: Mon May 1 16:13:40 2017 -0700 IMPALA-5262 : test_analytic_order_by_random fails with assert This was a poorly written test that relies on assumptions about the behavior of 'rand' and the order that rows get processed in a table that Impala doesn't actually guarantee. The new version is still sensitive to the precise behavior of 'rand()', but shouldn't be flaky unless that behavior is changed. Change-Id: If1ba8154c2b6a8d508916d85391b95885ef915a9 Reviewed-on: http://gerrit.cloudera.org:8080/6775 Reviewed-by: Alex Behm <alex.behm@cloudera.com> Tested-by: Impala Public Jenkins

          People

          • Assignee:
            twmarshall Thomas Tauber-Marshall
            Reporter:
            joemcdonnell Joe McDonnell
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development