Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12228 )
Change subject: IMPALA-8066: Build coordinator and executor containers ...................................................................... Patch Set 1: Code-Review+2 You think it's useful to have these because it's easier to think of a container as always using the same ports? Perhaps include that reasoning in the commit message. I would have naively thought that this is just configuration, as would kerberos or whatever be, but it's sensible to do it this way too. -- To view, visit http://gerrit.cloudera.org:8080/12228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I22f8ded167179478d7556f612b8b3e9d1b019a7a Gerrit-Change-Number: 12228 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Comment-Date: Thu, 17 Jan 2019 17:12:01 +0000 Gerrit-HasComments: No
