Re: Review Request 56685: Document security issue related to setting security.agent.hostname.validate to false

2017-02-14 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56685/#review165657 ---

Re: Review Request 56607: AMBARI-19915 Add Ranger KMS SSL properties in ambari stack

2017-02-14 Thread Mugdha Varadkar
> On Feb. 13, 2017, 7:18 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms.py, > > line 148 > > > > > > Today, we have symlinks from

Review Request 56699: AMBARI-20007:Args value is not retained for ssh node

2017-02-14 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56699/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 56622: AMBARI-19993 : Update atlas log-4j.xml configurations in ambari

2017-02-14 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56622/#review165644 --- Ship it! Ship It! - Sumit Mohanty On Feb. 13, 2017, 10:37

Re: Review Request 56608: AMBARI-19995. Improve logging for HSI's llapstatus command polling.

2017-02-14 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56608/#review165620 --- Ship it! Ship It! - Sumit Mohanty On Feb. 15, 2017, 1:49

Re: Review Request 55817: AMBARI-19618 Make cohosted components configurable in metainfo.xml instead of hardcoding in UI

2017-02-14 Thread yao lei
> On 二月 14, 2017, 3:50 a.m., Matt wrote: > > I'm curious whether this feature allow cohosting of another service's > > components? > > > > Eg: > > Service A has components A1 and A2 > > Service B has components B1, B2 and B3 > > Can A1 be coHosted with B2, using this feature? Thanks for your

Re: Review Request 56661: HiveView2.0: Table list under DB panel on Query tab appears only if clicked on db name and Not clear how to set the database

2017-02-14 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56661/#review165612 ---

Re: Review Request 56669: Ambari Views do not appear in full screen view

2017-02-14 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56669/#review165611 --- Ship it! Ship It! - Gaurav Nagar On Feb. 14, 2017, 5:42

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Bharat Viswanadham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56677/ --- (Updated Feb. 15, 2017, 12:25 a.m.) Review request for Ambari, Alejandro

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Alejandro Fernandez
> On Feb. 14, 2017, 11:02 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/configuration/kafka-broker.xml, > > line 157 > > > > > > During a fresh install this is

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Bharat Viswanadham
> On Feb. 14, 2017, 11:02 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/configuration/kafka-broker.xml, > > line 157 > > > > > > During a fresh install this is

Re: Review Request 56597: Externalize the logic for making navigation bar interactive

2017-02-14 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56597/#review165592 --- Ship it! Ship It! - Jaimin Jetly On Feb. 13, 2017, 11:50

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Bharat Viswanadham
> On Feb. 14, 2017, 11:02 p.m., Alejandro Fernandez wrote: > > This property is updated in Kafka 0.8.1 https://issues.apache.org/jira/browse/KAFKA-1113 So, during EU/RU this property should be deleted. (Here to this version of Kafka means, is it to latest version of Kafka in later HDP

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56677/#review165588 ---

Re: Review Request 56418: Export Blueprints does not contain the settings object and hence the credential store values

2017-02-14 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56418/#review165585 --- Ship it! Ship It! - Robert Nettleton On Feb. 9, 2017, 11:14

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/#review165584 --- Ship it! Ship It! - Robert Nettleton On Feb. 14, 2017, 7:06

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Bharat Viswanadham
> On Feb. 14, 2017, 8:39 p.m., Alejandro Fernandez wrote: > > But when some one does fresh install of HDP 2.6 this property should not be there correct? So, if i still have this log.cleanup.interval.mins 10 The frequency in minutes that the log cleaner checks whether any log

Review Request 56685: Document security issue related to setting security.agent.hostname.validate to false

2017-02-14 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56685/ --- Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene Chekanskiy,

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/#review165571 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 14, 2017,

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56677/#review165570 ---

Review Request 56679: Hosts page moving through pages gets progressively slower to the point of being unusable

2017-02-14 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56679/ --- Review request for Ambari, Aleksandr Kovalenko, Jaimin Jetly, and Yusaku Sako.

Review Request 56678: PixieDust - Decrease Service Check running time under 3 mins

2017-02-14 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56678/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Nate Cole.

Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-14 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/#review165566 --- Ship it! Ship It! - Di Li On Feb. 14, 2017, 12:37 a.m.,

Re: Review Request 56636: When yarn-site.xml is changed, MapReduce related components should be required to restart

2017-02-14 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56636/#review165564 --- Ship it! Ship It! - Di Li On Feb. 14, 2017, 3 a.m., Yuanbo

Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-02-14 Thread Bharat Viswanadham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56677/ --- Review request for Ambari and Alejandro Fernandez. Repository: ambari

Re: Review Request 56661: HiveView2.0: Table list under DB panel on Query tab appears only if clicked on db name and Not clear how to set the database

2017-02-14 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56661/#review165558 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 14, 2017, 1:59

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Oliver Szabo
> On Feb. 14, 2017, 6:44 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/solr_cloud_util.py, > > line 270 > > > > > > My point was that domain variable is

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/ --- (Updated Feb. 14, 2017, 7:06 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 56673: Perform Kinit on Kafka Start

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56673/#review165554 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 14, 2017,

Re: Review Request 56636: When yarn-site.xml is changed, MapReduce related components should be required to restart

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56636/#review165552 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 14, 2017, 3

Re: Review Request 56673: Perform Kinit on Kafka Start

2017-02-14 Thread Bharat Viswanadham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56673/ --- (Updated Feb. 14, 2017, 6:58 p.m.) Review request for Ambari and Juanjo

Re: Review Request 56669: Ambari Views do not appear in full screen view

2017-02-14 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56669/#review165549 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 14, 2017, 5:42

Re: Review Request 56607: AMBARI-19915 Add Ranger KMS SSL properties in ambari stack

2017-02-14 Thread Alejandro Fernandez
> On Feb. 13, 2017, 7:18 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms.py, > > line 148 > > > > > > Today, we have symlinks from

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/#review165546 ---

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/ --- (Updated Feb. 14, 2017, 6:40 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Oliver Szabo
> On Feb. 14, 2017, 6:32 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/solr_cloud_util.py, > > line 283 > > > > > > Make this robust, if solr_hosts is

Re: Review Request 56641: Change Livy recovery folder permission to 0700

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56641/#review165544 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 14, 2017,

Re: Review Request 56659: Add "Manage alert notifications" authorization

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56659/#review165542 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 14, 2017,

Re: Review Request 53686: Stage and Request status should be persisted in the database

2017-02-14 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53686/#review165540 --- Ship it! Ship It! - Nate Cole On Feb. 6, 2017, 6:09 p.m.,

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/#review165538 ---

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-14 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/ --- (Updated Feb. 14, 2017, 6:19 p.m.) Review request for Ambari, Jonathan Hurley,

Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-14 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/ --- Review request for Ambari, Miklos Gergely, Mugdha Varadkar, and Robert

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-14 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/#review165534 ---

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-14 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56599/#review165529 --- Ship it! Ship It! - Robert Levas On Feb. 14, 2017, 9:38

Re: Review Request 56540: Implement new DB checks for Postgres to prevent cross-schema confusion

2017-02-14 Thread Balázs Bence Sári
> On Feb. 14, 2017, 4:54 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java, > > line 780 > > > > > > I thought Logger uses {} for the

Re: Review Request 56540: Implement new DB checks for Postgres to prevent cross-schema confusion

2017-02-14 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56540/#review165527 --- Fix it, then Ship it!

Re: Review Request 56540: Implement new DB checks for Postgres to prevent cross-schema confusion

2017-02-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56540/#review165523 --- Fix it, then Ship it!

Re: Review Request 56231: AMBARI-19829: Several HDFS/YARN widgets on Heatmaps show N/A

2017-02-14 Thread Qin Liu
> On Feb. 13, 2017, 11:53 p.m., Aravindan Vijayan wrote: > > Ship It! Thank you so much Aravindan! - Qin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56231/#review165406

Re: Review Request 56659: Add "Manage alert notifications" authorization

2017-02-14 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56659/#review165520 --- Ship it! Ship It! - Robert Levas On Feb. 14, 2017, 7:21

Re: Review Request 56540: Implement new DB checks for Postgres to prevent cross-schema confusion

2017-02-14 Thread Balázs Bence Sári
> On Feb. 10, 2017, 1:38 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > line 714 > > > > > > Doc. Method is deleted. - Balázs Bence

Re: Review Request 56540: Implement new DB checks for Postgres to prevent cross-schema confusion

2017-02-14 Thread Balázs Bence Sári
> On Feb. 10, 2017, 2:18 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > lines 715-729 > > > > > > This is strange - why not make the

Re: Review Request 56643: Hive View 2.0 upload table issues : char support, endline support and temp file deletion

2017-02-14 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56643/#review165518 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 14, 2017,

Re: Review Request 56656: Log Search should not hide external Solr button even if it's mandatory

2017-02-14 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56656/#review165519 --- Ship it! Ship It! - Robert Nettleton On Feb. 14, 2017,

Re: Review Request 56540: Implement new DB checks for Postgres to prevent cross-schema confusion

2017-02-14 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56540/ --- (Updated Feb. 14, 2017, 3:27 p.m.) Review request for Ambari, Attila

Re: Review Request 56641: Change Livy recovery folder permission to 0700

2017-02-14 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56641/#review165515 --- Ship it! Ship It! - Sumit Mohanty On Feb. 14, 2017, 5:29

Re: Review Request 56663: Critical alert: 'Percent JournalNodes Available' appears after upgrade from ambari 2.2.2.0 to 2.5.0.0

2017-02-14 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56663/#review165513 --- Ship it! Ship It! - Vitalyi Brodetskyi On Лют. 14, 2017,

Re: Review Request 56663: Critical alert: 'Percent JournalNodes Available' appears after upgrade from ambari 2.2.2.0 to 2.5.0.0

2017-02-14 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56663/ --- (Updated Фев. 14, 2017, 3:13 п.п.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 56664: JS error in the "Supervisors Live" widget

2017-02-14 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56664/#review165510 --- Ship it! Ship It! - Denys Buzhor On Feb. 14, 2017, 2:43

Review Request 56664: JS error in the "Supervisors Live" widget

2017-02-14 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56664/ --- Review request for Ambari and Denys Buzhor. Bugs: AMBARI-20011

Re: Review Request 56663: Critical alert: 'Percent JournalNodes Available' appears after upgrade from ambari 2.2.2.0 to 2.5.0.0

2017-02-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56663/#review165508 --- Fix it, then Ship it!

Re: Review Request 56660: AMBARI-20005:Credentials from one workflow are getting associated across other workflow

2017-02-14 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56660/#review165503 --- Ship it! Ship It! - Nitiraj Rathore On Feb. 14, 2017, 1:06

Re: Review Request 56661: HiveView2.0: Table list under DB panel on Query tab appears only if clicked on db name and Not clear how to set the database

2017-02-14 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56661/ --- (Updated Feb. 14, 2017, 1:59 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-14 Thread Jonathan Hurley
> On Feb. 13, 2017, 10:49 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/PermissionEntity.java, > > lines 243-251 > > > > > > Any reason you didn't use a Set here

Review Request 56661: HiveView2.0: Table list under DB panel on Query tab appears only if clicked on db name and Not clear how to set the database

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

Re: Review Request 56607: AMBARI-19915 Add Ranger KMS SSL properties in ambari stack

2017-02-14 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56607/ --- (Updated Feb. 14, 2017, 1:18 p.m.) Review request for Ambari, Alejandro

Review Request 56660: AMBARI-20005:Credentials from one workflow are getting associated across other workflow

2017-02-14 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56660/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 56648: AMBARI-20001.Workflow manager- preview workflow has xml encodings

2017-02-14 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56648/#review165496 --- Ship it! Ship It! - Gaurav Nagar On Feb. 14, 2017, 9:39

Re: Review Request 56647: AMBARI-20000.Workflow Manager optimising finding nodes

2017-02-14 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56647/#review165495 --- Ship it! Ship It! - Gaurav Nagar On Feb. 14, 2017, 9:13

Review Request 56659: Add "Manage alert notifications" authorization

2017-02-14 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56659/ --- Review request for Ambari, Dmitro Lisnichenko, Robert Levas, and Vitalyi

Re: Review Request 56656: Log Search should not hide external Solr button even if it's mandatory

2017-02-14 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56656/#review165494 --- Ship it! Ship It! - Oliver Szabo On Feb. 14, 2017, 11:49

Review Request 56656: Log Search should not hide external Solr button even if it's mandatory

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

Re: Review Request 56653: UI on Admin page-> Auto Start tab is not consistent

2017-02-14 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56653/#review165489 --- Ship it! Ship It! - Denys Buzhor On Feb. 14, 2017, 11:13

Review Request 56653: UI on Admin page-> Auto Start tab is not consistent

2017-02-14 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56653/ --- Review request for Ambari. Bugs: ambari-20002

Re: Review Request 56607: AMBARI-19915 Add Ranger KMS SSL properties in ambari stack

2017-02-14 Thread Mugdha Varadkar
> On Feb. 13, 2017, 7:18 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms.py, > > line 148 > > > > > > Today, we have symlinks from

Re: Review Request 56599: ambari-server upgrade is not idempotent

2017-02-14 Thread Attila Magyar
> On Feb. 13, 2017, 3:49 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/PermissionEntity.java, > > lines 243-251 > > > > > > Any reason you didn't use a Set here

Re: Review Request 56647: AMBARI-20000.Workflow Manager optimising finding nodes

2017-02-14 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56647/#review165479 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 14, 2017,

Review Request 56648: AMBARI-20001.Workflow manager- preview workflow has xml encodings

2017-02-14 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56648/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, Nitiraj Rathore, and

Re: Review Request 56643: Hive View 2.0 upload table issues : char support, endline support and temp file deletion

2017-02-14 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56643/#review165475 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 14, 2017, 5:56

Re: Review Request 56643: Hive View 2.0 upload table issues : char support, endline support and temp file deletion

2017-02-14 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56643/#review165473 --- Ship it! Ship It! - Gaurav Nagar On Feb. 14, 2017, 5:56