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


Ship it!




Ship It!

- Jaimin Jetly


On March 3, 2016, 6:11 p.m., Di Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/44346/
> -----------------------------------------------------------
> 
> (Updated March 3, 2016, 6:11 p.m.)
> 
> 
> Review request for Ambari and Jaimin Jetly.
> 
> 
> Bugs: AMBARI-15285
>     https://issues.apache.org/jira/browse/AMBARI-15285
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The dir name should be called quicklinks, with lower case "l".
> It's currently quickLinks (upper case L) in trunk code, causing the following 
> error during ambari server start up...
> 
> 03 Mar 2016 07:16:15,133 ERROR [main] QuickLinksConfigurationModule:64 - 
> Quick links file not found
> 03 Mar 2016 07:16:15,134 ERROR [main] QuickLinksConfigurationModule:74 - 
> Unable to parse quicklinks configuration fil
> e 
> /var/lib/ambari-server/resources/common-services/AMBARI_METRICS/0.1.0/quicklinks/quicklinks.json
> java.io.EOFException: No content to map to Object due to end of input
> 
> 
> The fix is to correct the name to be the default one quicklinks.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/quickLinks/quicklinks.json
>  1c276c0 
>   
> ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/quicklinks/quicklinks.json
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/44346/diff/
> 
> 
> Testing
> -------
> 
> tested by patching a cluster installed with trunk code.
> 
> 
> Thanks,
> 
> Di Li
> 
>

Reply via email to