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

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

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

IMPALA-14776 (part 1 followup): Fix TestParquetMaxPageHeader

Part 1 of IMPALA-14776 added a call to super.teardown_method() for
TestParquetMaxPageHeader. However, it got the order wrong. The
super().teardown_method() needs to come after TestParquetMaxPageHeader's
own teardown steps, as those steps need the built-in clients.
This moves the super().teardown_method() call to be after those
steps.

Testing:
 - Ran TestParquetMaxPageHeader

Change-Id: If66b82d7386cdf748674a5a146d6b2b31471331d
Reviewed-on: http://gerrit.cloudera.org:8080/24065
Reviewed-by: Michael Smith <[email protected]>
Tested-by: Michael Smith <[email protected]>


> Fix pytest ResourceWarnings
> ---------------------------
>
>                 Key: IMPALA-14776
>                 URL: https://issues.apache.org/jira/browse/IMPALA-14776
>             Project: IMPALA
>          Issue Type: Task
>          Components: Infrastructure, Test
>    Affects Versions: Impala 5.0.0
>            Reporter: Joe McDonnell
>            Assignee: Joe McDonnell
>            Priority: Critical
>
> After switching to pytest 6.2.5, it now outputs information about leaked 
> resources (sockets, open files, thread pools):
> {noformat}
> 18:15:34 
> authorization/test_ranger.py::TestRangerIndependent::test_grant_revoke_with_role["table_format:
>  text/none" | "protocol: hs2" | "exec_option: {'abort_on_error': 1, 
> 'batch_size': 0, 'disable_codegen': False, 'disable_codegen_rows_threshold': 
> 0, 'exec_single_node_rows_threshold': 0, 'num_nodes': 0, 'test_replan': 1}"]
> 18:15:34   
> /data0/jenkins/workspace/impala-cdw-master-staging-core-asan/repos/Impala/tests/common/impala_test_suite.py:951:
>  ResourceWarning: unclosed <socket.socket fd=47, 
> family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=0, 
> laddr=('127.0.0.1', 44380), raddr=('127.0.0.1', 21050)>{noformat}
> We should fix these warnings. In particular, tests that don't clean up 
> clients can cause there to be orphaned HS2 sessions that trigger 
> TestValidateMetrics.test_metrics_are_zero().



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to