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

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

Commit c83cfcbe4b3a605187740caff36b0d728047c200 in impala's branch 
refs/heads/dependabot/pip/infra/python/deps/sqlparse-0.5.4 from Zoltan 
Borok-Nagy
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=c83cfcbe4 ]

IMPALA-15036: Set HTTP port for minicluster Trino in a more robust way

Prior to this patch Impala used sed to replace property
"http-server.http.port" in Trino's config file. However, newer Trino
versions might not define that property, which means sed fail silently
then Trino binds the default port 8080.

This patch first removes property http-server.http.port if it exists,
then adds it with the desired value.

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


> Set HTTP port for minicluster Trino in a more robust way
> --------------------------------------------------------
>
>                 Key: IMPALA-15036
>                 URL: https://issues.apache.org/jira/browse/IMPALA-15036
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Test
>            Reporter: Zoltán Borók-Nagy
>            Assignee: Zoltán Borók-Nagy
>            Priority: Major
>             Fix For: Impala 5.0.0
>
>
> Currently Impala uses sed to replace property "http-server.http.port" in 
> Trino's config file.
> However, newer Trino versions might not define that property, which means sed 
> fail silently and Trino binds the default port 8080.



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