Re: Review Request 55133: AMBARI-19306: User is not able to validate/submit the workflow after browsing workflow path

2017-01-04 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55133/#review160564 --- Ship it! Ship It! - Pallav Kulshreshtha On Jan. 4, 2017, 7:1

Re: Review Request 55164: AMBARI-19352 : Designer should always pick latest version for an action node

2017-01-04 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55164/#review160563 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 5, 2017, 6:34 a

Re: Review Request 55164: AMBARI-19352 : Designer should always pick latest version for an action node

2017-01-04 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55164/#review160562 --- Ship it! Ship It! - Pallav Kulshreshtha On Jan. 4, 2017, 8:4

Review Request 55195: solr_cloud_util.py should update solr data in ZK if it has changed

2017-01-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55195/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs: AMBARI-1937

Re: Review Request 55107: LogSearch upgrade support 2.4 -> 2.5

2017-01-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55107/ --- (Updated Jan. 5, 2017, 3:02 a.m.) Review request for Ambari, Oliver Szabo and R

Review Request 55193: Executing ambari-server unit tests with JDK 1.8 results in unit test failures

2017-01-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55193/ --- Review request for Ambari, Eugene Chekanskiy, Jaimin Jetly, Jonathan Hurley, Las

Review Request 55188: AMBARI-19372. Disable FileCache update for parallel_execution

2017-01-04 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55188/ --- Review request for Ambari, Dmytro Sen, Sandor Magyari, Sumit Mohanty, and Sebast

Re: Review Request 55182: Add common log rotation settings to ranger, ranger-kms and falcon

2017-01-04 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55182/#review160532 --- Ship it! Ship It! - Sumit Mohanty On Jan. 4, 2017, 7:06 p.m.

Re: Review Request 55159: AMBARI-19348 - Version number is incorrect when upgrade wizard is opened from Upgrade History

2017-01-04 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55159/#review160530 --- Ship it! Ship It! - Xi Wang On Jan. 3, 2017, 5:36 p.m., Rich

Re: Review Request 55178: Investigate Changing the Default Container Policy in JPA From Vector to ArrayList

2017-01-04 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55178/#review160525 --- Ship it! Ship It! - Sebastian Toader On Jan. 4, 2017, 5:56 p

Re: Review Request 55176: Log Search: external authentication roles are hard coded

2017-01-04 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55176/ --- (Updated Jan. 4, 2017, 7:19 p.m.) Review request for Ambari, Miklos Gergely and

Review Request 55182: Add common log rotation settings to ranger, ranger-kms and falcon

2017-01-04 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55182/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-19368 https://iss

Re: Review Request 55163: ambai-web action to download all client configs of host

2017-01-04 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55163/#review160522 --- Ship it! Ship It! - Yusaku Sako On Jan. 4, 2017, 7:17 a.m.,

Re: Review Request 55153: Previous Upgrade In Progress Pre-Req Check Incorrectly Detects ABORTED Upgrade

2017-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55153/#review160511 --- Ship it! Ship It! - Nate Cole On Jan. 4, 2017, 12:19 p.m., J

Re: Review Request 55178: Investigate Changing the Default Container Policy in JPA From Vector to ArrayList

2017-01-04 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55178/#review160509 --- Ship it! Ship It! - Sid Wagle On Jan. 4, 2017, 4:56 p.m., Jo

Re: Review Request 55153: Previous Upgrade In Progress Pre-Req Check Incorrectly Detects ABORTED Upgrade

2017-01-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55153/ --- (Updated Jan. 4, 2017, 12:19 p.m.) Review request for Ambari, Dmitro Lisnichenk

Re: Review Request 55178: Investigate Changing the Default Container Policy in JPA From Vector to ArrayList

2017-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55178/#review160507 --- Ship it! Ship It! - Nate Cole On Jan. 4, 2017, 11:56 a.m., J

Review Request 55178: Investigate Changing the Default Container Policy in JPA From Vector to ArrayList

2017-01-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55178/ --- Review request for Ambari, Nate Cole, Sebastian Toader, and Sid Wagle. Bugs: AM

Re: Review Request 55129: [AMBARI-19337] Ambari has some spelling mistakes in YARN proxyuser properties in many places.

2017-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55129/#review160506 --- Ship it! Ship It! - Nate Cole On Jan. 3, 2017, 3:41 a.m., Ja

Re: Review Request 55153: Previous Upgrade In Progress Pre-Req Check Incorrectly Detects ABORTED Upgrade

2017-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55153/#review160504 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 55176: Log Search: external authentication roles are hard coded

2017-01-04 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55176/#review160499 --- Fix it, then Ship it! Looks fine, other than the minor issue be

Re: Review Request 55171: Setup correct authentication and authorization mechanism between Yarn and Zookeeper

2017-01-04 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55171/#review160498 --- Ship it! Ship It! - Laszlo Puskas On Jan. 4, 2017, 3:10 p.m.

Re: Review Request 55171: Setup correct authentication and authorization mechanism between Yarn and Zookeeper

2017-01-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55171/#review160496 --- Ship it! Ship It! - Robert Levas On Jan. 4, 2017, 10:10 a.m.

Re: Review Request 55171: Setup correct authentication and authorization mechanism between Yarn and Zookeeper

2017-01-04 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55171/#review160495 --- Ship it! Ship It! - Sebastian Toader On Jan. 4, 2017, 4:10 p

Re: Review Request 55171: Setup correct authentication and authorization mechanism between Yarn and Zookeeper

2017-01-04 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55171/ --- (Updated Jan. 4, 2017, 3:10 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 55176: Log Search: external authentication roles are hard coded

2017-01-04 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55176/#review160493 --- Ship it! Ship It! - Miklos Gergely On Jan. 4, 2017, 2:19 p.m

Review Request 55176: Log Search: external authentication roles are hard coded

2017-01-04 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55176/ --- Review request for Ambari, Miklos Gergely and Robert Nettleton. Bugs: AMBARI-19

Re: Review Request 55171: Setup correct authentication and authorization mechanism between Yarn and Zookeeper

2017-01-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55171/#review160491 --- Ship it! Ship It! ambari-agent/src/main/java/org/apache/ambar

Review Request 55174: Ranger policies not syncing for storm, kafka and knox

2017-01-04 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55174/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jaimin Jetly, Jayu

Re: Review Request 55175: Hive View 2.0 : add REST endpoint for renaming a table

2017-01-04 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55175/#review160489 --- Ship it! Ship It! - Gaurav Nagar On Jan. 4, 2017, 1:03 p.m.,

Re: Review Request 55175: Hive View 2.0 : add REST endpoint for renaming a table

2017-01-04 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55175/#review160488 --- Ship it! Ship It! - Rohit Choudhary On Jan. 4, 2017, 1:03 p.

Re: Review Request 55168: Change type of fields for Ambari SNMP alert trap

2017-01-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55168/#review160487 --- ambari-server/src/main/java/org/apache/ambari/server/notification

Re: Review Request 55171: Setup correct authentication and authorization mechanism between Yarn and Zookeeper

2017-01-04 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55171/#review160486 --- Ship it! Ship It! - Sebastian Toader On Jan. 4, 2017, 1:42 p

Review Request 55175: Hive View 2.0 : add REST endpoint for renaming a table

2017-01-04 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55175/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav Kulshreshtha,

Review Request 55173: Flume metrics can't show if hostname of flume agent is not lowercase

2017-01-04 Thread yao lei
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55173/ --- Review request for Ambari. Bugs: AMBARI-19357 https://issues.apache.org/jir

Re: Review Request 55172: Cover JournalNode HA wizard with unit tests

2017-01-04 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55172/#review160484 --- Ship it! Ship It! - Aleksandr Kovalenko On Янв. 4, 2017, 12:

Review Request 55172: Cover JournalNode HA wizard with unit tests

2017-01-04 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55172/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-19361 https

Review Request 55171: Setup correct authentication and authorization mechanism between Yarn and Zookeeper

2017-01-04 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55171/ --- Review request for Ambari, Attila Doroszlai, Laszlo Puskas, Robert Levas, and Se

Review Request 55170: UI elements misalignment in dashboard screen of WFD

2017-01-04 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55170/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, Pall

Re: Review Request 55167: AMBARI-19354 : Invalid message is shown after saving the workflow

2017-01-04 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55167/#review160482 --- Ship it! Ship It! - Nitiraj Rathore On Jan. 4, 2017, 9:25 a.

Re: Review Request 55169: UI to add config ...2.6/HIVE/configuration/hive-env.xml 'enable_heap_dump' config in smart configs page

2017-01-04 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55169/#review160481 --- Ship it! Ship It! - Andriy Babiichuk On Січ. 4, 2017, полуде

Review Request 55169: UI to add config ...2.6/HIVE/configuration/hive-env.xml 'enable_heap_dump' config in smart configs page

2017-01-04 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55169/ --- Review request for Ambari and Andriy Babiichuk. Bugs: AMBARI-19358 https://

Review Request 55168: Change type of fields for Ambari SNMP alert trap

2017-01-04 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55168/ --- Review request for Ambari. Bugs: AMBARI-19356 https://issues.apache.org/jir

Review Request 55167: AMBARI-19354 : Invalid message is shown after saving the workflow

2017-01-04 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55167/ --- Review request for Ambari, Gaurav Nagar, Nitiraj Rathore, and Pallav Kulshreshth

Review Request 55164: AMBARI-19352 : Designer should always pick latest version for an action node

2017-01-04 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55164/ --- Review request for Ambari, Gaurav Nagar, Nitiraj Rathore, and Pallav Kulshreshth