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

run-tests.py not compatible with python 2.6

    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: Infrastructure
    • Labels:
    • Environment:
      Python 2.6

      Description

      09:14:57 Traceback (most recent call last):
      09:14:57   File "/data/jenkins/workspace/impala-private-build-and-test/repos/Impala/tests/run-tests.py", line 190, in <module>
      09:14:57     print_metrics('connections')
      09:14:57   File "/data/jenkins/workspace/impala-private-build-and-test/repos/Impala/tests/run-tests.py", line 160, in print_metrics
      09:14:57     print "connections metrics for impalad at port {}:".format(port)
      09:14:57 ValueError: zero length field name in format
      

        Activity

        Hide
        jbapple Jim Apple added a comment -
        IMPALA-4742: Change "{}".format() to "{0}".format() for Py 2.6
        
        From the Python docs:
        
        "Changed in version 2.7: The positional argument specifiers can be
        omitted, so '{} {}' is equivalent to '{0} {1}'."
        
        http://gerrit.cloudera.org:8080/5401 used the newer form,
        "{}".format(). This change uses the older backwards-compatible
        compatible form.
        
        Change-Id: If78b9b4061ca191932ac5b0b14e0ee8951a9d4e8
        Reviewed-on: http://gerrit.cloudera.org:8080/5641
        Reviewed-by: Michael Brown <mikeb@cloudera.com>
        Reviewed-by: Tim Armstrong <tarmstrong@cloudera.com>
        Tested-by: Impala Public Jenkins
        
        Show
        jbapple Jim Apple added a comment - IMPALA-4742: Change "{}".format() to "{0}".format() for Py 2.6 From the Python docs: "Changed in version 2.7: The positional argument specifiers can be omitted, so '{} {}' is equivalent to '{0} {1}'." http://gerrit.cloudera.org:8080/5401 used the newer form, "{}".format(). This change uses the older backwards-compatible compatible form. Change-Id: If78b9b4061ca191932ac5b0b14e0ee8951a9d4e8 Reviewed-on: http://gerrit.cloudera.org:8080/5641 Reviewed-by: Michael Brown <mikeb@cloudera.com> Reviewed-by: Tim Armstrong <tarmstrong@cloudera.com> Tested-by: Impala Public Jenkins

          People

          • Assignee:
            jbapple Jim Apple
            Reporter:
            jbapple Jim Apple
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development