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



ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/configuration/hbase-env.xml
<https://reviews.apache.org/r/33497/#comment131799>

    This will introduce a regression. HBASE_REGIONSERVER_OPTS in HDP2.2 is set 
to the following and should be set to the same in HDP2.3
    
    export HBASE_REGIONSERVER_OPTS="$HBASE_REGIONSERVER_OPTS 
-Xmn{{regionserver_xmn_size}} -XX:CMSInitiatingOccupancyFraction=70  
-Xms{{regionserver_heapsize}} -Xmx{{regionserver_heapsize}} {% if 
hbase_max_direct_memory_size %} 
-XX:MaxDirectMemorySize={{hbase_max_direct_memory_size}}m {% endif %} 
-Djava.security.auth.login.config={{regionserver_jaas_config_file}}"



ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/configuration/hbase-env.xml
<https://reviews.apache.org/r/33497/#comment131801>

    This is a regression. Same explanation as above. 
    
    Correct value should be.
    
    export HBASE_REGIONSERVER_OPTS="$HBASE_REGIONSERVER_OPTS 
-Xmn{{regionserver_xmn_size}} -XX:CMSInitiatingOccupancyFraction=70  
-Xms{{regionserver_heapsize}} -Xmx{{regionserver_heapsize}} {% if 
hbase_max_direct_memory_size %} 
-XX:MaxDirectMemorySize={{hbase_max_direct_memory_size}}m {% endif %}"


- Jayush Luniya


On April 24, 2015, 12:25 a.m., nick dimiduk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/33497/
> -----------------------------------------------------------
> 
> (Updated April 24, 2015, 12:25 a.m.)
> 
> 
> Review request for Ambari and Jayush Luniya.
> 
> 
> Bugs: AMBARI-10712
>     https://issues.apache.org/jira/browse/AMBARI-10712
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> per JIRA.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py
>  311819a 
>   
> ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/configuration/hbase-env.xml
>  PRE-CREATION 
>   ambari-server/src/test/python/stacks/2.0.6/configs/secured.json c54d2b0 
>   ambari-server/src/test/python/stacks/2.3/configs/hbase_secure.json 8580db6 
> 
> Diff: https://reviews.apache.org/r/33497/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> nick dimiduk
> 
>

Reply via email to