[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4438 To unsubscribe, v

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. The comments on https://issues

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4438 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: Verified-1 Build failed: http://sandbox.jenkins.cloudera.com/job/impala-external-ge

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: Code-Review+2 Looks good to me. -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: Code-Review+1 > Let me run the Leopard front end and do a sanity test. It's fine. +1 f

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: Let me run the Leopard front end and do a sanity test. -- To view, visit http://gerri

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4438/1/www/bootstrap/js/npm.js File www/bo

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4438/1/www/bootstrap/js/npm.js Fil

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4438/1/www/bootstrap/js/npm.js File www/bo

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4438/1/www/bootstrap/js/npm.js Fil

[Impala-ASF-CR] IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2.

2016-09-16 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/4438 Change subject: IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. .. IMPALA-4110, IMPALA-3853: npm.js uses Artistic License 2. The c