Review Request 46398: Name in create Alert Notification should be restricted to use only alphanumerics, hyphens, spaces and underscores

2016-04-19 Thread Anita Jebaraj
est/controllers/main/alerts/manage_alert_notifications_controller_test.js 837c2f3 Diff: https://reviews.apache.org/r/46398/diff/ Testing --- Ran mvn clean test 25660 tests complete (29 seconds) 154 tests pending Thanks, Anita Jebaraj

Re: Review Request 46398: Name in create Alert Notification should be restricted to use only alphanumerics, hyphens, spaces and underscores

2016-04-25 Thread Anita Jebaraj
, visit: https://reviews.apache.org/r/46398/#review129717 --- On April 19, 2016, 6:20 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. T

Review Request 47514: Kerberos wizard gets reset and does not remember selections/data entered in previous step when you click on Back

2016-05-17 Thread Anita Jebaraj
ros/step1_controller_test.js 280f923 Diff: https://reviews.apache.org/r/47514/diff/ Testing --- Ran mvn test 27719 tests complete (32 seconds) 154 tests pending Thanks, Anita Jebaraj

Re: Review Request 47514: Kerberos wizard gets reset and does not remember selections/data entered in previous step when you click on Back

2016-05-18 Thread Anita Jebaraj
web/app/views/main/admin/kerberos/step1_view.js 2e7c4cf ambari-web/test/controllers/main/admin/kerberos/step1_controller_test.js fda2ab4 Diff: https://reviews.apache.org/r/47514/diff/ Testing --- Ran mvn test 27719 tests complete (32 seconds) 154 tests pending Thanks, Anita Jebaraj

Re: Review Request 47514: Kerberos wizard gets reset and does not remember selections/data entered in previous step when you click on Back

2016-05-19 Thread Anita Jebaraj
g/r/47514/#review133814 --- On May 18, 2016, 7:05 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 47514: Kerberos wizard gets reset and does not remember selections/data entered in previous step when you click on Back

2016-06-17 Thread Anita Jebaraj
://reviews.apache.org/r/47514/#review138222 --- On May 18, 2016, 7:05 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > h

Review Request 49287: After switching to external database in hive, user should be allowed to delete mysql server

2016-06-27 Thread Anita Jebaraj
eb/test/models/stack_service_component_test.js 7e971ce ambari-web/test/views/main/host/details/host_component_view_test.js 648f0f6 Diff: https://reviews.apache.org/r/49287/diff/ Testing --- Ran mvn test 28974 tests complete (43 seconds) 154 tests pending Thanks, Anita Jebaraj

Re: Review Request 49287: After switching to external database in hive, user should be allowed to delete mysql server

2016-06-30 Thread Anita Jebaraj
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49287/#review140121 ----------- On June 27, 2016, 11:20 p.m., Anita Jebaraj wrote: > > -

Re: Review Request 49287: After switching to external database in hive, user should be allowed to delete mysql server

2016-07-01 Thread Anita Jebaraj
iew140362 --- On June 27, 2016, 11:20 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-14 Thread Anita Jebaraj
/common/test_stack_advisor.py a6baeea Diff: https://reviews.apache.org/r/50047/diff/ Testing --- Added 1 new test case, Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-19 Thread Anita Jebaraj
-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review142546 ----------- On July 14, 2016, 9:08 p.m., Anita Jebaraj wrote: > > --

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-19 Thread Anita Jebaraj
ndant... > > > > `kafka-broker/listeners` = `kafka-broker/listeners` > > Anita Jebaraj wrote: > Hi Robert, I had to include the declaration for listeners in > kerberos.json since the value for listeners wont be displayed in the UI under > the kerberos confi

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-20 Thread Anita Jebaraj
/0.9.0/kerberos.json 2b1c01b ambari-server/src/test/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelperTest.java ee2a671 Diff: https://reviews.apache.org/r/50047/diff/ Testing --- Added 1 new test case, Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-21 Thread Anita Jebaraj
ndant... > > > > `kafka-broker/listeners` = `kafka-broker/listeners` > > Anita Jebaraj wrote: > Hi Robert, I had to include the declaration for listeners in > kerberos.json since the value for listeners wont be displayed in the UI under > the kerberos confi

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-21 Thread Anita Jebaraj
------ On July 20, 2016, 10:01 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50047/ > ---

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-25 Thread Anita Jebaraj
/0.9.0/kerberos.json 2b1c01b ambari-server/src/test/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelperTest.java ee2a671 Diff: https://reviews.apache.org/r/50047/diff/ Testing --- Added 1 new test case, Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-25 Thread Anita Jebaraj
; > Why is 'SASL' hardcoded here? > > Anita Jebaraj wrote: > When kerberos is enabled, the listeners protocol can be PLAINTEXTSASL or > SASL_PLAINTEXT, in HDP PLAINTEXTSASL is used throughout, but the user can > still customize the protocol as SASL_PLAINTEXT, so I

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
d e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review143440 --- On July 25, 2016, 9:12 p.m., Anita Jebaraj wrote: > > --- > This is an automatically genera

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
> On July 25, 2016, 10:21 p.m., Robert Levas wrote: > > Ship It! > > Robert Levas wrote: > Thanks for making those changes. > > Anita Jebaraj wrote: > can you please help in pushing the code > > Robert Levas wrote: > sure... workin

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
> On July 25, 2016, 10:21 p.m., Robert Levas wrote: > > Ship It! > > Robert Levas wrote: > Thanks for making those changes. > > Anita Jebaraj wrote: > can you please help in pushing the code > > Robert Levas wrote: > sure... workin

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
e is modified with a bad value by the user, rather than handling it with a default value. Please give your comments, I will update the patch accordingly. Thanks in advance - Anita --- This is an automatically generated e-mail. To

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
Thanks, Anita Jebaraj

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
y, visit: https://reviews.apache.org/r/50047/#review143538 ------- On July 26, 2016, 6:15 p.m., Anita Jebaraj wrote: > > --- > This is an automatically gene

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
e > found during testing. It is unlikely users will use this facility when > setting the Kerberos descriptor... and if they do, the error should be made > ovbious to them, rather than hide the issue with some "default" value. > > Anita Jebaraj wrote: > Hi N

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
> On July 25, 2016, 10:21 p.m., Robert Levas wrote: > > Ship It! > > Robert Levas wrote: > Thanks for making those changes. > > Anita Jebaraj wrote: > can you please help in pushing the code > > Robert Levas wrote: > sure... workin

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
iew143599 --- On July 26, 2016, 6:15 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-07-26 Thread Anita Jebaraj
> On July 26, 2016, 8:41 p.m., Robert Levas wrote: > > Committed to trunk > > ``` > > commit 124f48ef899fddb6bdb96ebea9aa3a6a1a6adbca > > Author: Anita Jebaraj > > Date: Tue Jul 26 16:25:30 2016 -0400 > > ``` > > &

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-03 Thread Anita Jebaraj
eners protocol is updated and kafka started successfully Thanks, Anita Jebaraj

Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-17 Thread Anita Jebaraj
--- KAFKA is spelled as 'KAKFA' in stack_advisor.py because of which the function validateKAFKAConfigurations will not be called Diffs - Diff: https://reviews.apache.org/r/51189/diff/ Testing --- Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-17 Thread Anita Jebaraj
/resources/stacks/HDP/2.3/services/stack_advisor.py a0cce7b Diff: https://reviews.apache.org/r/51189/diff/ Testing --- Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-17 Thread Anita Jebaraj
iff: https://reviews.apache.org/r/51189/diff/ Testing --- Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-18 Thread Anita Jebaraj
iew146078 --- On Aug. 18, 2016, 5:45 a.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 51189: Typo in stack_advisor.py for KAFKA

2016-08-18 Thread Anita Jebaraj
> On Aug. 18, 2016, 2:05 p.m., Robert Levas wrote: > > Ship It! > > Anita Jebaraj wrote: > Can you please help in pushing the code? > > Robert Levas wrote: > Committed to trunk > ``` > commit 712a12b6b3228d5bd532aa7a594544c8bd594d66 >

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-18 Thread Anita Jebaraj
atch. - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review146131 --- On Aug. 3, 2016, 8:47 p.m., Anit

Review Request 51254: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-19 Thread Anita Jebaraj
got reset to default PLAINTEXT://localhost:6667 Sumit, please let me know if any other scenarios need to be tested. Thanks, Anita Jebaraj

Re: Review Request 51254: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-19 Thread Anita Jebaraj
Disabled kerberos, value got reset to default PLAINTEXT://localhost:6667 Sumit, please let me know if any other scenarios need to be tested. File Attachments (updated) validation.jpg https://reviews.apache.org/media/uploaded/files/2016/08/19/286c8651-2954-41ba-8d

Re: Review Request 51254: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-19 Thread Anita Jebaraj
ploaded/files/2016/08/19/286c8651-2954-41ba-8d53-b734129ff1a8__validation.jpg Thanks, Anita Jebaraj

Re: Review Request 51254: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-19 Thread Anita Jebaraj
ode in UpgradeCatalog240 for now and move it later or should I remove the UpgrageCatalog240 code. - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51254/#review146225 --

Re: Review Request 51254: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-19 Thread Anita Jebaraj
deCatalog240 is not the best option for trunk. It should be > > UpgradeCatalog for the next release number. Otherwise, upgrade from 240 to > > next release will miss these changes. > > > > You can open a JIRA to resolve it once next UpgradeCatalog is > > ide

Re: Review Request 51254: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-22 Thread Anita Jebaraj
ploaded/files/2016/08/19/286c8651-2954-41ba-8d53-b734129ff1a8__validation.jpg Thanks, Anita Jebaraj

Re: Review Request 51254: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

2016-08-22 Thread Anita Jebaraj
- On Aug. 22, 2016, 4:42 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51254/ > --

Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-09-07 Thread Anita Jebaraj
is masked. Thanks, Anita Jebaraj

Re: Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-09-07 Thread Anita Jebaraj
, the password fields in the configurations.json is masked. During testing, everytime the ambari agent is restarted, it registers with the server and the memory cache and file cache are updated, the alerts in turn uses the value from the memory cache. Thanks, Anita Jebaraj

Re: Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-09-12 Thread Anita Jebaraj
, the password fields in the configurations.json is masked. During testing, everytime the ambari agent is restarted, it registers with the server and the memory cache and file cache are updated, the alerts in turn uses the value from the memory cache. Thanks, Anita Jebaraj

Re: Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-09-12 Thread Anita Jebaraj
------- On Sept. 12, 2016, 8:20 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51705/ > ---

Re: Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-09-12 Thread Anita Jebaraj
air is not required in this case. I have updated the code - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51705/#review148212 -

Re: Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-09-12 Thread Anita Jebaraj
, the password fields in the configurations.json is masked. During testing, everytime the ambari agent is restarted, it registers with the server and the memory cache and file cache are updated, the alerts in turn uses the value from the memory cache. Thanks, Anita Jebaraj

Re: Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-09-13 Thread Anita Jebaraj
the files in the agent side by creating new JIRAs. - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51705/#review148705 -------

Review Request 52430: Include an option to download the saved query in hive view

2016-09-30 Thread Anita Jebaraj
://reviews.apache.org/r/52430/diff/ Testing --- Ran mvn test Added 1 new test case Thanks, Anita Jebaraj

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-09-30 Thread Anita Jebaraj
screenshot1.jpg https://reviews.apache.org/media/uploaded/files/2016/09/30/b494489d-90f6-42f1-847e-ac19ffcb817a__screenshot1.jpg Thanks, Anita Jebaraj

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-09-30 Thread Anita Jebaraj
-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52430/#review151068 --- On Sept. 30, 2016, 4:42 p.m., Anita Jebaraj wrote: > > --

Re: Review Request 51705: Password in the configurations.json file in the ambari-agent cache is not encrypted

2016-10-02 Thread Anita Jebaraj
matically generated e-mail. To reply, visit: https://reviews.apache.org/r/51705/#review151074 --- On Sept. 12, 2016, 10:48 p.m., Anita Jebaraj wrote: > > --- > This is an automat

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-10-03 Thread Anita Jebaraj
-ee6c0d3ddcfd__screenshot2.jpg screenshot1.jpg https://reviews.apache.org/media/uploaded/files/2016/09/30/b494489d-90f6-42f1-847e-ac19ffcb817a__screenshot1.jpg Thanks, Anita Jebaraj

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-10-03 Thread Anita Jebaraj
e download operation will happen when ?download=true, the fileName is also added as query param along with download - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-10-03 Thread Anita Jebaraj
To reply, visit: https://reviews.apache.org/r/52430/#review151070 ------- On Oct. 3, 2016, 6:13 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-10-03 Thread Anita Jebaraj
-ee6c0d3ddcfd__screenshot2.jpg screenshot1.jpg https://reviews.apache.org/media/uploaded/files/2016/09/30/b494489d-90f6-42f1-847e-ac19ffcb817a__screenshot1.jpg Thanks, Anita Jebaraj

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-10-03 Thread Anita Jebaraj
gt; > Ran mvn test > > Added 1 new test case > > > File Attachments > > > screenshot2.jpg > > https://reviews.apache.org/media/uploaded/files/2016/09/30/26f21b91-2637-4127-9ff2-ee6c0d3ddcfd__screenshot2.jpg > screenshot1.jpg > > https://reviews.apache.org/media/uploaded/files/2016/09/30/b494489d-90f6-42f1-847e-ac19ffcb817a__screenshot1.jpg > > > Thanks, > > Anita Jebaraj > >

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-10-04 Thread Anita Jebaraj
it: https://reviews.apache.org/r/52430/#review151374 --- On Oct. 3, 2016, 11:25 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 52430: Include an option to download the saved query in hive view

2016-10-07 Thread Anita Jebaraj
it: https://reviews.apache.org/r/52430/#review151812 --- On Oct. 7, 2016, 4:33 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To rep

Review Request 53126: Workflow Designer View: When you execute a workflow designer view instance, it does not load immediately unless you return to the dashboard.

2016-10-23 Thread Anita Jebaraj
anager/src/main/resources/view.xml ec3d5f5 Diff: https://reviews.apache.org/r/53126/diff/ Testing --- Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 53126: Workflow Designer View: When you execute a workflow designer view instance, it does not load immediately unless you return to the dashboard.

2016-10-23 Thread Anita Jebaraj
view names. Diffs - contrib/views/wfmanager/src/main/resources/view.xml ec3d5f5 Diff: https://reviews.apache.org/r/53126/diff/ Testing --- Ran mvn test Thanks, Anita Jebaraj

Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
/ui/hive-web/app/templates/index.hbs a06c330 Diff: https://reviews.apache.org/r/53140/diff/ Testing --- Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
-f60e-4250-84d5-25987617fa3d__screenshot1.jpg screenshot2.jpg https://reviews.apache.org/media/uploaded/files/2016/10/24/9cc85ddf-8554-4db1-a132-f1380017c7d3__screenshot2.jpg Thanks, Anita Jebaraj

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
g/r/53140/#review153724 --- On Oct. 24, 2016, 4:58 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 53140: Include an option to upload query in hive-view

2016-10-24 Thread Anita Jebaraj
> On Oct. 24, 2016, 6:57 p.m., Alexandr Antonenko wrote: > > Ship It! > > Anita Jebaraj wrote: > Thank you...Can you please help in pushing the code? > > Alexandr Antonenko wrote: > done Thank you - Anita ---

Re: Review Request 53126: Workflow Designer View: When you execute a workflow designer view instance, it does not load immediately unless you return to the dashboard.

2016-10-27 Thread Anita Jebaraj
ssue still exists and the patch is relevant. Thanks, Anita Jebaraj

Re: Review Request 53126: Workflow Designer View: When you execute a workflow designer view instance, it does not load immediately unless you return to the dashboard.

2016-10-27 Thread Anita Jebaraj
his is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53126/#review154063 --- On Oct. 27, 2016, 6:39 p.m., Anita Jebaraj wrote: > > --- > This is an autom

Re: Review Request 53126: Workflow Designer View: When you execute a workflow designer view instance, it does not load immediately unless you return to the dashboard.

2016-10-27 Thread Anita Jebaraj
> On Oct. 27, 2016, 6:49 p.m., DIPAYAN BHOWMICK wrote: > > Ship It! > > Anita Jebaraj wrote: > Hi Dipayan, Can you please help in pushing the code. This is an isolated > change and hence wont impact the changes from Ambari-18691 > > DIPAYAN BHOWMICK wrote: &g

Review Request 53273: Not able to change the default port ambari server listens to

2016-10-28 Thread Anita Jebaraj
Diff: https://reviews.apache.org/r/53273/diff/ Testing --- Performed manual testing. Thanks, Anita Jebaraj

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-10-31 Thread Anita Jebaraj
l. To reply, visit: https://reviews.apache.org/r/53273/#review154267 ----------- On Oct. 28, 2016, 9:46 p.m., Anita Jebaraj wrote: > > --- > This is an automati

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-10-31 Thread Anita Jebaraj
/src/main/python/ambari_server/serverConfiguration.py 766f4cc ambari-server/src/main/python/ambari_server/utils.py 5886561 Diff: https://reviews.apache.org/r/53273/diff/ Testing --- Performed manual testing. Thanks, Anita Jebaraj

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-10-31 Thread Anita Jebaraj
manual testing. Thanks, Anita Jebaraj

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-01 Thread Anita Jebaraj
can you please help in pushing the code? - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53273/#review154216 ----------- On Nov. 1, 2016, 5:39 a.m., Anita Jebaraj wrote: > > ---

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-02 Thread Anita Jebaraj
/src/main/python/ambari_server/serverConfiguration.py 766f4cc ambari-server/src/main/python/ambari_server/utils.py 348d5df ambari-server/src/test/python/TestUtils.py 5d32125 Diff: https://reviews.apache.org/r/53273/diff/ Testing --- Performed manual testing. Thanks, Anita Jebaraj

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-02 Thread Anita Jebaraj
DIR: /etc/ambari-server/conf, exit code 1' > > ``` The test case test_wait_for_pid has passed in Hadoop QA and I also tried running it using python unitTests.py and it passed - Anita --- This is an automatically

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-07 Thread Anita Jebaraj
/src/main/python/ambari_server/serverConfiguration.py 766f4cc ambari-server/src/main/python/ambari_server/utils.py 757c333 ambari-server/src/test/python/TestUtils.py 367b7fe Diff: https://reviews.apache.org/r/53273/diff/ Testing --- Performed manual testing. Thanks, Anita Jebaraj

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-07 Thread Anita Jebaraj
n automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53273/#review155164 --- On Nov. 7, 2016, 6:11 p.m., Anita Jebaraj wrote: > > --- > This is an automa

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-08 Thread Anita Jebaraj
ly, visit: https://reviews.apache.org/r/53273/#review155299 ------- On Nov. 7, 2016, 6:11 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 53273: Not able to change the default port ambari server listens to

2016-11-08 Thread Anita Jebaraj
ranch-2.5, AMBARI-18741-Nov7.patch does > > not apply to branch-2.5 > > Anita Jebaraj wrote: > Hi Di, these changes are not included in branch-2.5, this issue was > existing only in trunk. Thank you for pushing the changes - Anita ---

Review Request 53788: HTTP responses needs to have the character encoding specified in the content type header

2016-11-15 Thread Anita Jebaraj
test. The test cases failing in hadoop QA is not related to the patch for this jira Thanks, Anita Jebaraj

Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-02-28 Thread Anita Jebaraj
/configuration/ConfigurationTest.java 51114f8 Diff: https://reviews.apache.org/r/57168/diff/ Testing --- Added 1 test case Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-02-28 Thread Anita Jebaraj
test case Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-01 Thread Anita Jebaraj
://reviews.apache.org/r/57168/diff/3/ Changes: https://reviews.apache.org/r/57168/diff/2-3/ Testing --- Added 1 test case Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-01 Thread Anita Jebaraj
nce the jira is accepted I can write a wiki about how to use this feature. - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57168/#review167543 --------

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-02 Thread Anita Jebaraj
ted e-mail. To reply, visit: https://reviews.apache.org/r/57168/#review167286 --- On March 2, 2017, 1:05 a.m., Anita Jebaraj wrote: > > --- > This is an automatically generate

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-03 Thread Anita Jebaraj
://reviews.apache.org/r/57168/diff/4/ Changes: https://reviews.apache.org/r/57168/diff/3-4/ Testing --- Added 1 test case Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-03 Thread Anita Jebaraj
> > general setting. Fixed in new patch - Anita ------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57168/#review167838 --- On March 3, 2017, 8:47 p.m., Anita Jebaraj wrote: > > -

Review Request 57346: Value for "User Limit Factor" should be float instead of integer in YARN Queue Manager.

2017-03-06 Thread Anita Jebaraj
/templates/queue.hbs 69f5b3b Diff: https://reviews.apache.org/r/57346/diff/1/ Testing --- Tested manually File Attachments screenshot-1.png https://reviews.apache.org/media/uploaded/files/2017/03/06/f55e8f25-b625-4285-96ee-beb4fa6992a4__screenshot-1.png Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-06 Thread Anita Jebaraj
://reviews.apache.org/r/57168/diff/5/ Changes: https://reviews.apache.org/r/57168/diff/4-5/ Testing --- Added 1 test case Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-06 Thread Anita Jebaraj
ps://reviews.apache.org/r/57168/#review168045 ------- On March 6, 2017, 10:45 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 57346: Value for "User Limit Factor" should be float instead of integer in YARN Queue Manager.

2017-03-06 Thread Anita Jebaraj
media/uploaded/files/2017/03/06/f55e8f25-b625-4285-96ee-beb4fa6992a4__screenshot-1.png Thanks, Anita Jebaraj

Re: Review Request 57346: Value for "User Limit Factor" should be float instead of integer in YARN Queue Manager.

2017-03-06 Thread Anita Jebaraj
nshot-1.png https://reviews.apache.org/media/uploaded/files/2017/03/06/f55e8f25-b625-4285-96ee-beb4fa6992a4__screenshot-1.png Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-08 Thread Anita Jebaraj
/org/apache/ambari/server/configuration/ConfigurationTest.java 51114f8 Diff: https://reviews.apache.org/r/57168/diff/6/ Changes: https://reviews.apache.org/r/57168/diff/5-6/ Testing --- Added 1 test case Ran mvn test Thanks, Anita Jebaraj

Review Request 57467: After clearing the content of Users in Yarn Queue Manager, it still shows the user in effective administrators

2017-03-09 Thread Anita Jebaraj
inistraters. Diffs - contrib/views/capacity-scheduler/src/main/resources/ui/app/components/userGroupInput.js 513031c Diff: https://reviews.apache.org/r/57467/diff/1/ Testing --- Tested manually Thanks, Anita Jebaraj

Re: Review Request 57467: After clearing the content of Users in Yarn Queue Manager, it still shows the user in effective administrators

2017-03-09 Thread Anita Jebaraj
Before Fix https://reviews.apache.org/media/uploaded/files/2017/03/09/6ca6a48d-cde2-4e1b-a209-aa898244c998__beforefix.png After fix https://reviews.apache.org/media/uploaded/files/2017/03/09/062bb336-0706-442d-a638-f07f8f8363b3__afterFix.PNG Thanks, Anita Jebaraj

Re: Review Request 57467: After clearing the content of Users in Yarn Queue Manager, it still shows the user in effective administrators

2017-03-09 Thread Anita Jebaraj
g/r/57467/#review168505 --- On March 9, 2017, 5:24 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 57346: Value for "User Limit Factor" should be float instead of integer in YARN Queue Manager.

2017-03-09 Thread Anita Jebaraj
g/r/57346/#review168506 --- On March 8, 2017, 11:15 p.m., Anita Jebaraj wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-10 Thread Anita Jebaraj
/org/apache/ambari/server/configuration/ConfigurationTest.java 8111a39 Diff: https://reviews.apache.org/r/57168/diff/7/ Changes: https://reviews.apache.org/r/57168/diff/6-7/ Testing --- Added 1 test case Ran mvn test Thanks, Anita Jebaraj

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-10 Thread Anita Jebaraj
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57168/#review168610 ----------- On March 10, 2017, 3:34 p.m., Anita Jebaraj wrote: > > -

Re: Review Request 57168: Include option to filter out properties from APi that returns ambari.properties file

2017-03-10 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57168/#review168320 ------- On March 10, 2017, 3:34 p.m., Anita Jebaraj wrote: > > --

  1   2   >