[ 
https://issues.apache.org/jira/browse/IMPALA-8247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16778242#comment-16778242
 ] 

ASF subversion and git services commented on IMPALA-8247:
---------------------------------------------------------

Commit e9936b02cf5b9673d6dd8f3624b07e2ec81f763e in impala's branch 
refs/heads/master from Joe McDonnell
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=e9936b0 ]

IMPALA-8247: Fix tests missing from unified backend executable

When converting the tests in be/src/util to the unified backend
test executable in IMPALA-8071, some tests did not get linked
into the executable appropriately. This means that they stopped
running. Specifically, the tests in bit-stream-utils-test.cc
and system-state-info-test.cc were left out.

This adds them back and modifies bin/validate-unified-backend-filters.py
to check both directions:
 - If a filter is specified, it must match a test in the executable.
 - If the executable has a test, some filter must match it.
I ran the backend tests on debug and ASAN with this change.

Change-Id: I80d8b5779cb0ac663e32c72722e0c73b68a43d2e
Reviewed-on: http://gerrit.cloudera.org:8080/12584
Tested-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Tim Armstrong <[email protected]>


> Backend tests from bit-stream-utils-test.cc and system-state-info-test.cc are 
> not running
> -----------------------------------------------------------------------------------------
>
>                 Key: IMPALA-8247
>                 URL: https://issues.apache.org/jira/browse/IMPALA-8247
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Backend, Infrastructure
>    Affects Versions: Impala 3.2.0
>            Reporter: Joe McDonnell
>            Assignee: Joe McDonnell
>            Priority: Blocker
>              Labels: broken-build
>
> Several tests in be/src/util were converted to the unified backend test 
> executable in IMPALA-8071. Unfortunately, some tests stopped running as part 
> of this change. Specifically, the tests in bit-stream-utils-test.cc and 
> system-state-info-test.cc were not included in the test binary and stopped 
> running. The tests changed to use ADD_UNIFIED_BE_LSAN_TEST, but the files 
> were not included in the UtilTests library.
> These two files should be included in the test executable and we need to 
> update the validation of the unified test executable 
> (bin/validate-unified-backend-test-filters.py) to catch this case.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to