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

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

Commit 2fad3d08dd6d6c4ebc27f56d8a03e637a65fd29c in impala's branch 
refs/heads/master from Michael Brown
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=2fad3d0 ]

IMPALA-8171: minicluster: use exec builtin to start Impala daemons

In IMPALA-7779 bin/start-daemon.sh was written to unify the place where
Impala daemons are started. It had a side-effect of keeping itself alive
as a parent process of the actual daemon. This broke the
tests.comparison.cluster abstraction used by the stress test and others
in that the process detection logic was finding false positives.

The old bin/start-*.sh scripts used the exec builtin to call their
daemons. Resurrect this as a simple fix; it also makes the ps list
shorter. Add a system test to catch this in the future.

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


> stress test doesn't work against minicluster
> --------------------------------------------
>
>                 Key: IMPALA-8171
>                 URL: https://issues.apache.org/jira/browse/IMPALA-8171
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Infrastructure
>    Affects Versions: Impala 3.2.0
>            Reporter: Michael Brown
>            Assignee: Michael Brown
>            Priority: Critical
>
> {noformat}
>  $ tests/stress/concurrent_select.py --tpch-db tpch_parquet
> Traceback (most recent call last):
>   File "tests/stress/concurrent_select.py", line 2320, in <module>
>     main()
>   File "tests/stress/concurrent_select.py", line 2159, in main
>     if impala.find_stopped_impalads():
>   File "/home/mikeb/Impala/tests/comparison/cluster.py", line 557, in 
> find_stopped_impalads
>     for idx, pid in enumerate(self.for_each_impalad(lambda i: i.find_pid())):
>   File "/home/mikeb/Impala/tests/comparison/cluster.py", line 611, in 
> for_each_impalad
>     results = promise.get(maxint)
>   File "/usr/lib/python2.7/multiprocessing/pool.py", line 567, in get
>     raise self._value
> ValueError: invalid literal for int() with base 10: '48601\n48624\n'
> {noformat}
> This is due to IMPALA-7999. The refactor causes additional processes to match 
> the process list.



--
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