[
https://issues.apache.org/jira/browse/IMPALA-7828?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16679473#comment-16679473
]
ASF subversion and git services commented on IMPALA-7828:
---------------------------------------------------------
Commit d1aa1c009f62500ae2ee8cd915751b0d42bee911 in impala's branch
refs/heads/master from Michael Ho
[ https://git-wip-us.apache.org/repos/asf?p=impala.git;h=d1aa1c0 ]
IMPALA-7828: A temporary workaround for flaky UDF test (test_mem_leak())
Before IMPALA-4063, the error message detected during
FragmentInstanceState::Close() was always lost. After
IMPALA-4063, we may sometimes get the error message in
FragmentInstanceState::Close(). It's non-deterministic
as the fragment instance thread may race with the query
state thread which reports the final status. The UDF test
currently tries to handle this non-determinism by using
"row_regex:.*" in the ERRORS section but it doesn't
always seem to work.
This change workarounds the issue by commenting out the
ERRORS section in udf-no-expr-rewrite.text for now.
The actual fix will be done in IMPALA-7829.
Change-Id: I6a55d5ad1a5a7278e7390f60854a8df28c1b9f28
Reviewed-on: http://gerrit.cloudera.org:8080/11900
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> test_mem_leak() is flaky
> ------------------------
>
> Key: IMPALA-7828
> URL: https://issues.apache.org/jira/browse/IMPALA-7828
> Project: IMPALA
> Issue Type: Bug
> Components: Backend
> Affects Versions: Impala 3.2.0
> Reporter: Michael Ho
> Assignee: Michael Ho
> Priority: Blocker
> Labels: broken-build
>
> Before IMPALA-4063, the error message detected during
> {{FragmentInstanceState::Close()}} was always lost. After IMPALA-4063, we may
> sometimes get the error message in {{FragmentInstanceState::Close()}}. It's
> non-deterministic as the fragment instance thread may race with the query
> state thread which reports the final status. The test currently tries to
> handle this non-determinism by using "row_regex:.*" in the {{--ERRORS--}}
> section but it doesn't seem to work.
> Let's unbreak the test for now. Longer run, we need to move the point in
> which all fragment instances are done after
> {{FragmentInstanceState::Close()}} so things will become deterministic. This
> may have implication to query completion time so performance may need to be
> evaluated.
> {noformat}
> Stacktrace
> query_test/test_udfs.py:313: in test_ir_functions
> self.run_test_case('QueryTest/udf-no-expr-rewrite', vector,
> use_db=unique_database)
> common/impala_test_suite.py:496: in run_test_case
> self.__verify_results_and_errors(vector, test_section, result, use_db)
> common/impala_test_suite.py:358: in __verify_results_and_errors
> replace_filenames_with_placeholder)
> common/test_result_verifier.py:362: in verify_raw_results
> verify_errors(expected_errors, actual_errors)
> common/test_result_verifier.py:314: in verify_errors
> VERIFIER_MAP['VERIFY_IS_EQUAL'](expected, actual)
> common/test_result_verifier.py:271: in verify_query_result_is_equal
> assert expected_results == actual_results
> E assert Comparing QueryTestResults (expected vs actual):
> E row_regex:.* != None
> E Number of rows returned (expected vs actual): 1 != 0
> {noformat}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
