You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by Tom Beerbower <tb...@hortonworks.com> on 2016/04/21 15:36:21 UTC

Review Request 46489: Atlas Integration : Make default Atlas configuration for embedded Hbase

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

Review request for atlas, John Speidel and Nate Cole.


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


Repository: ambari


Description
-------

Even though the standalone Atlas uses embedded Hbase by default, the Ambari Atlas configuration is set for Berkeley DB. Use embedded Hbase when installing Atlas through Ambari.


Diffs
-----

  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml a6deda1 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-env.xml d64d48c 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-hbase-site.xml PRE-CREATION 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 133b264 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py 041e04e 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py 3443016 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/atlas-env.xml 42503b5 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/metainfo.xml c787b40 
  ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py fd3ca81 
  ambari-server/src/test/python/stacks/2.3/configs/default.json f87a4b8 
  ambari-server/src/test/python/stacks/2.3/configs/secure.json 524484f 
  ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py a882189 
  ambari-server/src/test/python/stacks/2.5/configs/default.json ddf4c9a 

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


Testing
-------

Manual tested with Atlas install.

Unit tests updated.

mvn clean test


Thanks,

Tom Beerbower


Re: Review Request 46489: Atlas Integration : Make default Atlas configuration for embedded Hbase

Posted by Tom Beerbower <tb...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46489/
-----------------------------------------------------------

(Updated April 22, 2016, 1:57 a.m.)


Review request for atlas, John Speidel and Nate Cole.


Changes
-------

updated diff


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


Repository: ambari


Description
-------

Even though the standalone Atlas uses embedded Hbase by default, the Ambari Atlas configuration is set for Berkeley DB. Use embedded Hbase when installing Atlas through Ambari.


Diffs (updated)
-----

  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml a6deda1 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-env.xml d64d48c 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-hbase-site.xml PRE-CREATION 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 133b264 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py 041e04e 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py 3443016 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/atlas-env.xml 42503b5 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/metainfo.xml c787b40 
  ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py fd3ca81 
  ambari-server/src/test/python/stacks/2.3/configs/default.json f87a4b8 
  ambari-server/src/test/python/stacks/2.3/configs/secure.json 524484f 
  ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py a882189 
  ambari-server/src/test/python/stacks/2.5/configs/default.json ddf4c9a 

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


Testing
-------

Manual tested with Atlas install.

Unit tests updated.

mvn clean test


Thanks,

Tom Beerbower


Re: Review Request 46489: Atlas Integration : Make default Atlas configuration for embedded Hbase

Posted by Tom Beerbower <tb...@hortonworks.com>.

> On April 21, 2016, 9:07 p.m., Nate Cole wrote:
> > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py, line 67
> > <https://reviews.apache.org/r/46489/diff/1/?file=1354958#file1354958line67>
> >
> >     older deployments won't have this value - should it be defaulted (like stack_name ablove)?

Yes, thanks for catching that and thanks for the review!


- Tom


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


On April 22, 2016, 1:57 a.m., Tom Beerbower wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46489/
> -----------------------------------------------------------
> 
> (Updated April 22, 2016, 1:57 a.m.)
> 
> 
> Review request for atlas, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-15981
>     https://issues.apache.org/jira/browse/AMBARI-15981
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Even though the standalone Atlas uses embedded Hbase by default, the Ambari Atlas configuration is set for Berkeley DB. Use embedded Hbase when installing Atlas through Ambari.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml a6deda1 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-env.xml d64d48c 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-hbase-site.xml PRE-CREATION 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 133b264 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py 041e04e 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py 3443016 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/atlas-env.xml 42503b5 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/metainfo.xml c787b40 
>   ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py fd3ca81 
>   ambari-server/src/test/python/stacks/2.3/configs/default.json f87a4b8 
>   ambari-server/src/test/python/stacks/2.3/configs/secure.json 524484f 
>   ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py a882189 
>   ambari-server/src/test/python/stacks/2.5/configs/default.json ddf4c9a 
> 
> Diff: https://reviews.apache.org/r/46489/diff/
> 
> 
> Testing
> -------
> 
> Manual tested with Atlas install.
> 
> Unit tests updated.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>


Re: Review Request 46489: Atlas Integration : Make default Atlas configuration for embedded Hbase

Posted by Nate Cole <nc...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46489/#review129958
-----------------------------------------------------------


Ship it!





ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py (line 67)
<https://reviews.apache.org/r/46489/#comment193521>

    older deployments won't have this value - should it be defaulted (like stack_name ablove)?


- Nate Cole


On April 21, 2016, 9:36 a.m., Tom Beerbower wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46489/
> -----------------------------------------------------------
> 
> (Updated April 21, 2016, 9:36 a.m.)
> 
> 
> Review request for atlas, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-15981
>     https://issues.apache.org/jira/browse/AMBARI-15981
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Even though the standalone Atlas uses embedded Hbase by default, the Ambari Atlas configuration is set for Berkeley DB. Use embedded Hbase when installing Atlas through Ambari.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml a6deda1 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-env.xml d64d48c 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-hbase-site.xml PRE-CREATION 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 133b264 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py 041e04e 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py 3443016 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/atlas-env.xml 42503b5 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/metainfo.xml c787b40 
>   ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py fd3ca81 
>   ambari-server/src/test/python/stacks/2.3/configs/default.json f87a4b8 
>   ambari-server/src/test/python/stacks/2.3/configs/secure.json 524484f 
>   ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py a882189 
>   ambari-server/src/test/python/stacks/2.5/configs/default.json ddf4c9a 
> 
> Diff: https://reviews.apache.org/r/46489/diff/
> 
> 
> Testing
> -------
> 
> Manual tested with Atlas install.
> 
> Unit tests updated.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>