Thanks for fixing this. I’ll take a look in about an hour from now. Dimitris
On Thu, Mar 15, 2018 at 6:49 PM Alex Behm (Code Review) <[email protected]> wrote: > Alex Behm *posted comments* on this change. > > View Change <http://gerrit.cloudera.org:8080/9684> > > Patch set 1:Code-Review +1 > > Looks good to me. I think we should add a custom cluster test to cover > this scenario, but not necessarily now. > > > To view, visit change 9684 <http://gerrit.cloudera.org:8080/9684>. To > unsubscribe, visit settings <http://gerrit.cloudera.org:8080/settings>. > Gerrit-Project: Impala-ASF > Gerrit-Branch: master > Gerrit-MessageType: comment > Gerrit-Change-Id: I1ee6c6477458e0f4dd31b12daa9ed5f146d84e7b > Gerrit-Change-Number: 9684 > Gerrit-PatchSet: 1 > Gerrit-Owner: Tianyi Wang <[email protected]> > Gerrit-Reviewer: Alex Behm <[email protected]> > Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> > Gerrit-Reviewer: Tianyi Wang <[email protected]> > Gerrit-Comment-Date: Fri, 16 Mar 2018 01:49:30 +0000 > Gerrit-HasComments: No >
