ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vitalyi Brodetskyi" <vbrodets...@hortonworks.com>
Subject Re: Review Request 37953: SQLA: add property validation check for BP oozie/hive, about SQLA should be available only for stack 2.3+
Date Tue, 01 Sep 2015 16:38:51 GMT


> On Вер. 1, 2015, 4:35 після полудня, Jayush Luniya wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidatorImpl.java,
line 107
> > <https://reviews.apache.org/r/37953/diff/1-2/?file=1060321#file1060321line107>
> >
> >     Shouldnt this be 2.3.2.0?

No. 2.3.2 it's repo version but not HDP. Today me and Sumit, taled with Bob Nettleton and
decided to do in this way. Repo version we can't get on BP validation phase, because we will
have it only after deploy started.


> On Вер. 1, 2015, 4:35 після полудня, Jayush Luniya wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidatorImpl.java,
line 118
> > <https://reviews.apache.org/r/37953/diff/1-2/?file=1060321#file1060321line118>
> >
> >     Same as above.

The same as above.


- Vitalyi


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


On Вер. 1, 2015, 3:40 після полудня, Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37953/
> -----------------------------------------------------------
> 
> (Updated Вер. 1, 2015, 3:40 після полудня)
> 
> 
> Review request for Ambari, Jayush Luniya, Myroslav Papirkovskyy, and Sumit Mohanty.
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> .
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidatorImpl.java
70d1907 
> 
> Diff: https://reviews.apache.org/r/37953/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message