Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24346 )
Change subject: IMPALA-15036: Set HTTP port for minicluster Trino in a more robust way ...................................................................... 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]> --- M testdata/bin/minicluster_trino/Dockerfile 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24346 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If8e1300694200aaaf675675fb348f0c767e6c147 Gerrit-Change-Number: 24346 Gerrit-PatchSet: 3 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
