Dan Hecht has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
......................................................................


Patch Set 2:

> > It passed the packaging build: 
> > http://golden.jenkins.cloudera.com/job/CDH5-Packaging-Impala-On-Demand/348/
 > >
 > > (posted the wrong link for this patch in my previous comment)
 > 
 > Dan, any other concerns about this or the related patch?

No. But does the packaging build actually run the product?  It'd probably be 
best to do that (or run BVT) if not to make sure the classes can be found.

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
Gerrit-HasComments: No

Reply via email to