Review Request 60358: Hivesevers's status shows issue for the thrift's port conflict, when hiveservers and spark thrift server installed in the same host.

2017-06-21 Thread zhangxiaolu zhangxiaolu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60358/ --- Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, Dmytro Sen,

Re: Review Request 57583: HDFS Bytes Written and HDFS Bytes Read maybe not correct

2017-06-21 Thread zhangxiaolu zhangxiaolu
> On 六月 9, 2017, 11:57 a.m., Dmytro Sen wrote: > > Ship It! thank you for reviewing this patch. Could you be so kind as to commit the patch for me please? Thanks in advance - zhangxiaolu --- This is an automatically generated e-mail.

Re: Review Request 60265: Grafana Yarn Queue dashboard is not working due to incorrect startTime.

2017-06-21 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60265/#review178569 --- Ship it! Ship It! - Vivek Ratnavel Subramanian On June 21,

Re: Review Request 60201: AMBARI-17382 : Migrate AMS queries to use ROW_TIMESTAMP instead of native timerange hint

2017-06-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60201/ --- (Updated June 21, 2017, 8:39 p.m.) Review request for Ambari, Dmytro Sen,

Re: Review Request 60249: AMBARI-21234 Ambari rack awareness for Kafka

2017-06-21 Thread Ambud Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60249/ --- (Updated June 21, 2017, 8:33 p.m.) Review request for Ambari and Alejandro

Re: Review Request 60201: AMBARI-17382 : Migrate AMS queries to use ROW_TIMESTAMP instead of native timerange hint

2017-06-21 Thread Aravindan Vijayan
> On June 21, 2017, 10:06 a.m., Dmytro Sen wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/query/PhoenixTransactSQL.java > > Line 45 (original) > >

Re: Review Request 60205: Add support for consecutive login failure accounting

2017-06-21 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60205/ --- (Updated June 21, 2017, 3:46 p.m.) Review request for Ambari, Attila Magyar,

Re: Review Request 60205: Add support for consecutive login failure accounting

2017-06-21 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60205/ --- (Updated June 21, 2017, 3:45 p.m.) Review request for Ambari, Attila Magyar,

Review Request 60277: Pausing An Upgrade In Progress Does Not Resume PENDING Tasks

2017-06-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60277/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-21298

Re: Review Request 60265: Grafana Yarn Queue dashboard is not working due to incorrect startTime.

2017-06-21 Thread Jay SenSharma
> On June 21, 2017, 4:21 p.m., Aravindan Vijayan wrote: > > LGTM. Can you add a UI enginner (Vivek Subramanian or Richard Zang) to the > > review? Added reviewer 'rzang, vivekratnavel' - Jay --- This is an automatically generated

Re: Review Request 60265: Grafana Yarn Queue dashboard is not working due to incorrect startTime.

2017-06-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60265/#review178503 --- Ship it! LGTM. Can you add a UI enginner (Vivek Subramanian or

Re: Review Request 60205: Add support for consecutive login failure accounting

2017-06-21 Thread Sebastian Toader
> On June 20, 2017, 1:45 p.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariAuthenticationEventHandlerImpl.java > > Lines 109-117 (patched) > > > > > >

Re: Review Request 60205: Add support for consecutive login failure accounting

2017-06-21 Thread Robert Levas
> On June 20, 2017, 7:45 a.m., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariAuthenticationEventHandlerImpl.java > > Lines 109-117 (patched) > > > > > >

Re: Review Request 60271: AMBARI-21295. Enable Swagger checkstyle rule for endpoints documented so far

2017-06-21 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60271/#review178494 --- Ship it! Ship It! - Balázs Bence Sári On June 21, 2017, 1

Re: Review Request 60270: AMBARI-21294. Setup should keep existing connection-pool setting in ambari.properties

2017-06-21 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60270/#review178492 ---

Re: Review Request 60269: AMBARI-21293. Allow skipping Swagger generation

2017-06-21 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60269/#review178491 --- Ship it! Ship It! - Balázs Bence Sári On June 21, 2017,

Re: Review Request 60271: AMBARI-21295. Enable Swagger checkstyle rule for endpoints documented so far

2017-06-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60271/#review178490 --- Ship it! Ship It! - Sebastian Toader On June 21, 2017, 3

Review Request 60271: AMBARI-21295. Enable Swagger checkstyle rule for endpoints documented so far

2017-06-21 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60271/ --- Review request for Ambari, Balázs Bence Sári, Jaimin Jetly, Sumit Mohanty, and

Re: Review Request 60248: Service creation requires a repository id, not a string version

2017-06-21 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60248/#review178489 --- Ship it! Can we create a Jira to remove cluster.addService()?

Re: Review Request 60248: Service creation requires a repository id, not a string version

2017-06-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60248/#review178488 --- Ship it! Ship It! - Dmitro Lisnichenko On June 21, 2017,

Re: Review Request 60265: Grafana Yarn Queue dashboard is not working due to incorrect startTime.

2017-06-21 Thread Jay SenSharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60265/ --- (Updated June 21, 2017, 12:35 p.m.) Review request for Ambari, Aravindan

Re: Review Request 60269: AMBARI-21293. Allow skipping Swagger generation

2017-06-21 Thread Attila Doroszlai
> On June 21, 2017, 1:05 p.m., Balázs Bence Sári wrote: > > What about the other steps (generating html, copying artifacts to > > ambar-web) should they also be skipped? * `swagger-codegen-maven-plugin` current version 2.2.2 has no way to skip execution, ability to skip via `codegen.skip`

Re: Review Request 60270: AMBARI-21294. Setup should keep existing connection-pool setting in ambari.properties

2017-06-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60270/#review178485 --- Ship it! Ship It! - Sebastian Toader On June 21, 2017, 1:38

Review Request 60270: AMBARI-21294. Setup should keep existing connection-pool setting in ambari.properties

2017-06-21 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60270/ --- Review request for Ambari, Jonathan Hurley and Sebastian Toader. Bugs:

Re: Review Request 60152: Add "zeppelin.server.ssl.port" in Zeppelin's setting

2017-06-21 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60152/#review178482 --- Ship it! Ship It! - Pallav Kulshreshtha On June 19, 2017,

Re: Review Request 60152: Add "zeppelin.server.ssl.port" in Zeppelin's setting

2017-06-21 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60152/#review178483 --- Ship it! Ship It! - DIPAYAN BHOWMICK On June 19, 2017, 7:10

Re: Review Request 60269: AMBARI-21293. Allow skipping Swagger generation

2017-06-21 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60269/#review178481 --- What about the other steps (generating html, copying artifacts to

Re: Review Request 60201: AMBARI-17382 : Migrate AMS queries to use ROW_TIMESTAMP instead of native timerange hint

2017-06-21 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60201/#review178480 ---

Re: Review Request 60196: Fail to create solr clients in Log Search / Log Feeder with openjdk8

2017-06-21 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60196/ --- (Updated June 21, 2017, 10:16 a.m.) Review request for Ambari, Miklos Gergely

Re: Review Request 60201: AMBARI-17382 : Migrate AMS queries to use ROW_TIMESTAMP instead of native timerange hint

2017-06-21 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60201/#review178479 ---

Re: Review Request 60201: AMBARI-17382 : Migrate AMS queries to use ROW_TIMESTAMP instead of native timerange hint

2017-06-21 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60201/#review178477 ---

Re: Review Request 60265: Grafana Yarn Queue dashboard is not working due to incorrect startTime.

2017-06-21 Thread Jay SenSharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60265/ --- (Updated June 21, 2017, 9:48 a.m.) Review request for Ambari, Aravindan

Re: Review Request 60201: AMBARI-17382 : Migrate AMS queries to use ROW_TIMESTAMP instead of native timerange hint

2017-06-21 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60201/#review178475 --- ambari-metrics/ambari-metrics-timelineservice/pom.xml Line 37

Re: Review Request 60269: AMBARI-21293. Allow skipping Swagger generation

2017-06-21 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60269/#review178473 --- Ship it! Ship It! - Laszlo Puskas On June 21, 2017, 9:24

Re: Review Request 60112: Namenode variable should be auto populated for FS action node

2017-06-21 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60112/#review178472 --- Ship it! Ship It! - Padma Priya N On June 21, 2017, 9:32

Re: Review Request 60129: Workflow submission fails when action node names contain white space

2017-06-21 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60129/#review178471 --- Ship it! Ship It! - Pallav Kulshreshtha On June 15, 2017, 7

Re: Review Request 60112: Namenode variable should be auto populated for FS action node

2017-06-21 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60112/ --- (Updated June 21, 2017, 9:32 a.m.) Review request for Ambari, belliraj hb,

Re: Review Request 60269: AMBARI-21293. Allow skipping Swagger generation

2017-06-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60269/#review178470 --- Ship it! Ship It! - Sebastian Toader On June 21, 2017,

Review Request 60269: AMBARI-21293. Allow skipping Swagger generation

2017-06-21 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60269/ --- Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, and Sebastian

Re: Review Request 60268: Convert Slider View to use yarn package manager

2017-06-21 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60268/#review178468 --- Ship it! Ship It! - belliraj hb On June 21, 2017, 8:56

Re: Review Request 60114: Remove YARN client mode option from WFM

2017-06-21 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60114/#review178467 --- Ship it! Ship It! - belliraj hb On June 20, 2017, 5:59

Re: Review Request 60114: Remove YARN client mode option from WFM

2017-06-21 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60114/#review178466 --- Ship it! Ship It! - Padma Priya N On June 20, 2017, 5:59

Re: Review Request 60268: Convert Slider View to use yarn package manager

2017-06-21 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60268/#review178465 --- Ship it! Ship It! - Padma Priya N On June 21, 2017, 8:56

Re: Review Request 60128: Integrate new hdfs-file browser in WFD

2017-06-21 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60128/#review178464 --- Ship it! Ship It! - Padma Priya N On June 21, 2017, 8:47

Review Request 60268: Convert Slider View to use yarn package manager

2017-06-21 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60268/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Padma Priya N, and Pallav

Re: Review Request 60128: Integrate new hdfs-file browser in WFD

2017-06-21 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60128/ --- (Updated June 21, 2017, 8:47 a.m.) Review request for Ambari, belliraj hb,

Review Request 60265: Grafana Yarn Queue dashboard is not working due to incorrect startTime.

2017-06-21 Thread Jay SenSharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60265/ --- Review request for Ambari, Aravindan Vijayan and Sid Wagle. Bugs: AMBARI-21290

Re: Review Request 60249: AMBARI-21234 Ambari rack awareness for Kafka

2017-06-21 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60249/#review178450 --- Fix it, then Ship it! Run unit ambari-server unit tests and