Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-05-02 Thread Alexander Denissov 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/#review131348
---


Ship it!




Ship It!

- Alexander Denissov


On April 29, 2016, 8:50 p.m., bhuvnesh chaudhary wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46852/
> ---
> 
> (Updated April 29, 2016, 8:50 p.m.)
> 
> 
> Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
> Diachenko.
> 
> 
> Bugs: AMBARI-16191
> https://issues.apache.org/jira/browse/AMBARI-16191
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> HAWQ Configuration should be updated whenever Namenode is being moved using 
> the move namenode wizard.
> Parameters which should be updated
> Case 1: HDFS HA
> hdfs-client parameters under HAWQ service
> dfs.namenode.rpc-address..
> dfs.namenode.http-address..
> Case 2: Non HA HDFS
> hawq-site parmeters under HAWQ service
> hawq_dfs_url
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/service/reassign/step4_controller.js 
> a20aa82 
>   ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
>   ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
> 460726b 
> 
> Diff: https://reviews.apache.org/r/46852/diff/
> 
> 
> Testing
> ---
> 
> yes.
>   27472 tests complete (29 seconds)
>   154 tests pending
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 46.556 s
> [INFO] Finished at: 2016-04-29T13:53:51-07:00
> [INFO] Final Memory: 14M/338M
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread jun aoki 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/#review131228
---


Ship it!




Ship It!

- jun aoki


On April 29, 2016, 8:50 p.m., bhuvnesh chaudhary wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46852/
> ---
> 
> (Updated April 29, 2016, 8:50 p.m.)
> 
> 
> Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
> Diachenko.
> 
> 
> Bugs: AMBARI-16191
> https://issues.apache.org/jira/browse/AMBARI-16191
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> HAWQ Configuration should be updated whenever Namenode is being moved using 
> the move namenode wizard.
> Parameters which should be updated
> Case 1: HDFS HA
> hdfs-client parameters under HAWQ service
> dfs.namenode.rpc-address..
> dfs.namenode.http-address..
> Case 2: Non HA HDFS
> hawq-site parmeters under HAWQ service
> hawq_dfs_url
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/service/reassign/step4_controller.js 
> a20aa82 
>   ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
>   ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
> 460726b 
> 
> Diff: https://reviews.apache.org/r/46852/diff/
> 
> 
> Testing
> ---
> 
> yes.
>   27472 tests complete (29 seconds)
>   154 tests pending
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 46.556 s
> [INFO] Finished at: 2016-04-29T13:53:51-07:00
> [INFO] Final Memory: 14M/338M
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary 


> On April 29, 2016, 9:12 p.m., jun aoki wrote:
> > ambari-web/app/utils/configs/move_namenode_config_initializer.js, line 113
> > 
> >
> > Did you test if this works when HAWQ is not installed?
> 
> bhuvnesh chaudhary wrote:
> Did not test it yet without HAWQ, will do. 
> However, it will work.

Tested no issues without hawq.


- bhuvnesh


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/#review131189
---


On April 29, 2016, 8:50 p.m., bhuvnesh chaudhary wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46852/
> ---
> 
> (Updated April 29, 2016, 8:50 p.m.)
> 
> 
> Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
> Diachenko.
> 
> 
> Bugs: AMBARI-16191
> https://issues.apache.org/jira/browse/AMBARI-16191
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> HAWQ Configuration should be updated whenever Namenode is being moved using 
> the move namenode wizard.
> Parameters which should be updated
> Case 1: HDFS HA
> hdfs-client parameters under HAWQ service
> dfs.namenode.rpc-address..
> dfs.namenode.http-address..
> Case 2: Non HA HDFS
> hawq-site parmeters under HAWQ service
> hawq_dfs_url
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/service/reassign/step4_controller.js 
> a20aa82 
>   ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
>   ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
> 460726b 
> 
> Diff: https://reviews.apache.org/r/46852/diff/
> 
> 
> Testing
> ---
> 
> yes.
>   27472 tests complete (29 seconds)
>   154 tests pending
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 46.556 s
> [INFO] Finished at: 2016-04-29T13:53:51-07:00
> [INFO] Final Memory: 14M/338M
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary 


> On April 29, 2016, 9:12 p.m., jun aoki wrote:
> > ambari-web/app/utils/configs/move_namenode_config_initializer.js, line 113
> > 
> >
> > Did you test if this works when HAWQ is not installed?

Did not test it yet without HAWQ, will do. 
However, it will work.


> On April 29, 2016, 9:12 p.m., jun aoki wrote:
> > ambari-web/app/controllers/main/service/reassign/step4_controller.js, line 
> > 571
> > 
> >
> > you can use serviceToConfigSiteMap structure to add HAWQ.

Yes i could have, but to keep in sync with the current codebase (HBASE and 
ACCUMULO service configuration are updated in case of NAMENODE move as well), 
so added it here. So will keep it as is.


- bhuvnesh


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/#review131189
---


On April 29, 2016, 8:50 p.m., bhuvnesh chaudhary wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46852/
> ---
> 
> (Updated April 29, 2016, 8:50 p.m.)
> 
> 
> Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
> Diachenko.
> 
> 
> Bugs: AMBARI-16191
> https://issues.apache.org/jira/browse/AMBARI-16191
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> HAWQ Configuration should be updated whenever Namenode is being moved using 
> the move namenode wizard.
> Parameters which should be updated
> Case 1: HDFS HA
> hdfs-client parameters under HAWQ service
> dfs.namenode.rpc-address..
> dfs.namenode.http-address..
> Case 2: Non HA HDFS
> hawq-site parmeters under HAWQ service
> hawq_dfs_url
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/service/reassign/step4_controller.js 
> a20aa82 
>   ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
>   ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
> 460726b 
> 
> Diff: https://reviews.apache.org/r/46852/diff/
> 
> 
> Testing
> ---
> 
> yes.
>   27472 tests complete (29 seconds)
>   154 tests pending
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 46.556 s
> [INFO] Finished at: 2016-04-29T13:53:51-07:00
> [INFO] Final Memory: 14M/338M
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary 


> On April 29, 2016, 9:16 p.m., jun aoki wrote:
> > ambari-web/app/controllers/main/service/reassign/step4_controller.js, line 
> > 83
> > 
> >
> > Do you know how related services (or HAWQ) are restared after 
> > proeprties are changed?

Map unrelatedServicesMap controls the services which will be started. In case 
of Namenode, since NAMENODE is not available as a key, it will result in 
defined, and in the function startRequiredServices, it will go ahead and start 
all services (similarly for stop)

  /**
   * make server call to start services
   */
  startRequiredServices


- bhuvnesh


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/#review131191
---


On April 29, 2016, 8:50 p.m., bhuvnesh chaudhary wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46852/
> ---
> 
> (Updated April 29, 2016, 8:50 p.m.)
> 
> 
> Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
> Diachenko.
> 
> 
> Bugs: AMBARI-16191
> https://issues.apache.org/jira/browse/AMBARI-16191
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> HAWQ Configuration should be updated whenever Namenode is being moved using 
> the move namenode wizard.
> Parameters which should be updated
> Case 1: HDFS HA
> hdfs-client parameters under HAWQ service
> dfs.namenode.rpc-address..
> dfs.namenode.http-address..
> Case 2: Non HA HDFS
> hawq-site parmeters under HAWQ service
> hawq_dfs_url
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/service/reassign/step4_controller.js 
> a20aa82 
>   ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
>   ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
> 460726b 
> 
> Diff: https://reviews.apache.org/r/46852/diff/
> 
> 
> Testing
> ---
> 
> yes.
>   27472 tests complete (29 seconds)
>   154 tests pending
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 46.556 s
> [INFO] Finished at: 2016-04-29T13:53:51-07:00
> [INFO] Final Memory: 14M/338M
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread jun aoki 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/#review131191
---




ambari-web/app/controllers/main/service/reassign/step4_controller.js (line 83)


Do you know how related services (or HAWQ) are restared after proeprties 
are changed?


- jun aoki


On April 29, 2016, 8:50 p.m., bhuvnesh chaudhary wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46852/
> ---
> 
> (Updated April 29, 2016, 8:50 p.m.)
> 
> 
> Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
> Diachenko.
> 
> 
> Bugs: AMBARI-16191
> https://issues.apache.org/jira/browse/AMBARI-16191
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> HAWQ Configuration should be updated whenever Namenode is being moved using 
> the move namenode wizard.
> Parameters which should be updated
> Case 1: HDFS HA
> hdfs-client parameters under HAWQ service
> dfs.namenode.rpc-address..
> dfs.namenode.http-address..
> Case 2: Non HA HDFS
> hawq-site parmeters under HAWQ service
> hawq_dfs_url
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/service/reassign/step4_controller.js 
> a20aa82 
>   ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
>   ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
> 460726b 
> 
> Diff: https://reviews.apache.org/r/46852/diff/
> 
> 
> Testing
> ---
> 
> yes.
>   27472 tests complete (29 seconds)
>   154 tests pending
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 46.556 s
> [INFO] Finished at: 2016-04-29T13:53:51-07:00
> [INFO] Final Memory: 14M/338M
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread jun aoki 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/#review131189
---




ambari-web/app/controllers/main/service/reassign/step4_controller.js (line 571)


you can use serviceToConfigSiteMap structure to add HAWQ.



ambari-web/app/utils/configs/move_namenode_config_initializer.js (line 113)


Did you test if this works when HAWQ is not installed?


- jun aoki


On April 29, 2016, 8:50 p.m., bhuvnesh chaudhary wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46852/
> ---
> 
> (Updated April 29, 2016, 8:50 p.m.)
> 
> 
> Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
> Diachenko.
> 
> 
> Bugs: AMBARI-16191
> https://issues.apache.org/jira/browse/AMBARI-16191
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> HAWQ Configuration should be updated whenever Namenode is being moved using 
> the move namenode wizard.
> Parameters which should be updated
> Case 1: HDFS HA
> hdfs-client parameters under HAWQ service
> dfs.namenode.rpc-address..
> dfs.namenode.http-address..
> Case 2: Non HA HDFS
> hawq-site parmeters under HAWQ service
> hawq_dfs_url
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/service/reassign/step4_controller.js 
> a20aa82 
>   ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
>   ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
> 460726b 
> 
> Diff: https://reviews.apache.org/r/46852/diff/
> 
> 
> Testing
> ---
> 
> yes.
>   27472 tests complete (29 seconds)
>   154 tests pending
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 46.556 s
> [INFO] Finished at: 2016-04-29T13:53:51-07:00
> [INFO] Final Memory: 14M/338M
> 
> 
> Thanks,
> 
> bhuvnesh chaudhary
> 
>

Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46852/
---

Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr 
Diachenko.


Bugs: AMBARI-16191
https://issues.apache.org/jira/browse/AMBARI-16191


Repository: ambari


Description
---

HAWQ Configuration should be updated whenever Namenode is being moved using the 
move namenode wizard.
Parameters which should be updated
Case 1: HDFS HA
hdfs-client parameters under HAWQ service
dfs.namenode.rpc-address..
dfs.namenode.http-address..
Case 2: Non HA HDFS
hawq-site parmeters under HAWQ service
hawq_dfs_url


Diffs
-

  ambari-web/app/controllers/main/service/reassign/step4_controller.js a20aa82 
  ambari-web/app/utils/configs/move_namenode_config_initializer.js 160848c 
  ambari-web/test/controllers/main/service/reassign/step4_controller_test.js 
460726b 

Diff: https://reviews.apache.org/r/46852/diff/


Testing
---

yes.
  27472 tests complete (29 seconds)
  154 tests pending
[INFO] 
[INFO] BUILD SUCCESS
[INFO] 
[INFO] Total time: 46.556 s
[INFO] Finished at: 2016-04-29T13:53:51-07:00
[INFO] Final Memory: 14M/338M


Thanks,

bhuvnesh chaudhary