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

Precommits are not running strict_hs2_protocol=True configurations for shell tests

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • Impala 4.1.0
    • Impala 4.2.0, Impala 4.1.1
    • Clients
    • None
    • ghx-label-2

    Description

      When debugging a shell test failure, I noticed that the set of tests running in precommit is different from what happens when I run in my development environment. In precommit, none of the strict_hs2_protocol=True shell tests run. I can't figure out why it behaves that way from the code, given that we enable this test dimension:

        def add_test_dimensions(cls):
          # Run with both beeswax and HS2 to ensure that behaviour is the same.
          cls.ImpalaTestMatrix.add_dimension(create_client_protocol_dimension())
          cls.ImpalaTestMatrix.add_dimension(create_client_protocol_strict_dimension())
          cls.ImpalaTestMatrix.add_constraint(lambda v:
                v.get_value('protocol') != 'beeswax' or not v.get_value('strict_hs2_protocol'))

      And create_client_protocol_strict_dimension() should produce both False and True:

      def create_client_protocol_strict_dimension():
        # only support strict dimensions if the file system is HDFS, since that is
        # where the hive cluster is run.
        if IS_HDFS:
          return ImpalaTestDimension('strict_hs2_protocol', False, True)
        else:
          return create_client_protocol_no_strict_dimension()

      When I run locally, those tests run (and there are some failures to fix).

      Example precommit job: https://jenkins.impala.io/job/ubuntu-16.04-from-scratch/16296/testReport/shell.test_shell_interactive/TestImpalaShellInteractive/

      Attachments

        Issue Links

          Activity

            People

              joemcdonnell Joe McDonnell
              joemcdonnell Joe McDonnell
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: