----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16820/#review31761 -----------------------------------------------------------
Ship it! Ship It! - Sumit Mohanty On Jan. 14, 2014, 4:24 p.m., John Speidel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16820/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2014, 4:24 p.m.) > > > Review request for Ambari, Mahadev Konar, Nate Cole, Sumit Mohanty, Sid > Wagle, and Tom Beerbower. > > > Bugs: AMBARI-4277 > https://issues.apache.org/jira/browse/AMBARI-4277 > > > Repository: ambari > > > Description > ------- > > Part of the work being done to implement blueprints. > As a result of the blueprint work, the stacks api will be exposed to users > and therefore needs to be updated to use the api framework to be consistent > with the other api endpoints. > > We currently have '/stacks' and '/stacks2' endpoints. The '/stacks' endpoint > is very old and doesn't use the api framework and is inconsistent with the > rest of the api. This endpoint isn't used by the UI or any other internal > component and has never been documented. The 'stacks2' endpoint does use the > api framework and is consistent with the other endpoints in it's usage except > for the naming of the endpoint and some of it's children. This endpoint is > currently used by the UI. > > This task will basically clone the functionality of the 'stacks2' endpoint in > the 'stacks' endpoint. The 'stacks' endpoint will be augmented with > additional information in future tasks as needed by blueprints. Examples are > component dependencies and topology tokens in config properties. After this > work the 'stacks2' endpoint should basically be deprecated and the UI should > eventually be phased over to the new 'stacks' endpoint. > > > Diffs > ----- > > > Diff: https://reviews.apache.org/r/16820/diff/ > > > Testing > ------- > > - Functional testing of patch > -- Both /stacks2 and /stacks endpoints work as expected > > - All unit tests > > ---------------------------------------------------------------------- > Ran 173 tests in 1.562s > > OK > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 17:28.211s > [INFO] Finished at: Mon Jan 13 11:13:13 EST 2014 > [INFO] Final Memory: 23M/337M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > John Speidel > >
