Details
-
Bug
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
Impala 3.1.0, Impala 3.2.0
-
ghx-label-7
Description
Build log output:
04:20:18 =================================== FAILURES =================================== 04:20:18 ___________________________ TestWebPage.test_catalog ___________________________ 04:20:18 [gw4] linux2 -- Python 2.7.5 /data/jenkins/workspace/impala-cdh6.x-core/repos/Impala/bin/../infra/python/env/bin/python 04:20:18 webserver/test_web_pages.py:265: in test_catalog 04:20:18 self.__test_table_metrics("functional", "alltypes", "total-file-size-bytes") 04:20:18 webserver/test_web_pages.py:283: in __test_table_metrics 04:20:18 "?name=%s.%s" % (db_name, tbl_name), metric, ports_to_test=self.CATALOG_TEST_PORT) 04:20:18 webserver/test_web_pages.py:144: in get_and_check_status 04:20:18 assert response.status_code == requests.codes.ok\ 04:20:18 E AssertionError: Offending url: http://localhost:25020/table_metrics?name=functional.alltypes 04:20:18 E assert (200 == 200 and 'total-file-size-bytes' in '<!--\nLicensed to the Apache Software Foundation (ASF) under one\nor more contributor license agreements. See the NOTI...lltypesare not available because the table is currently modified by another operation.</pre>\n\n</div>\n</body>\n</html>\n\n') 04:20:18 E + where 200 = <Response [200]>.status_code 04:20:18 E + and 200 = <lookup 'status_codes'>.ok 04:20:18 E + where <lookup 'status_codes'> = requests.codes 04:20:18 E + and '<!--\nLicensed to the Apache Software Foundation (ASF) under one\nor more contributor license agreements. See the NOTI...lltypesare not available because the table is currently modified by another operation.</pre>\n\n</div>\n</body>\n</html>\n\n' = <Response [200]>.text
Hard to read, but apparently the test wants to perform two checks at once:
assert response.status_code == requests.codes.ok\ and string_to_search in response.text, "Offending url: " + input_url
The return code part appears to match. So the question is whether the string "total-file-size-bytes" appears in the output.
Tried building from latest master and hitting my local server with the same URL as above. The page reported:
num-partitions: 24 total-file-size-bytes: 489934
So, the string match part should work.
General observation: it is not helpful at all to include an and in an assert statement: doing so makes debugging very difficult. Please use one condition per assert.
Net result is that there is no obvious failure here; someone will need to did into it more deeply.