ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matt" <mithm...@gmail.com>
Subject Re: Review Request 42155: AMBARI-14605 '[RAM_DISK]' configured in dfs.datanode.data.dir fails to validate
Date Tue, 12 Jan 2016 05:47:25 GMT


> On Jan. 11, 2016, 8:59 p.m., Matt wrote:
> > I do not have much context on this one.
> > 
> > Does the validation fail only because of the underscore in [RAM_DISK]?
> 
> yao lei wrote:
>     Yes, dfs.datanode.data.dir should accept the underscore in [RAM_DISK].
>     If remove the underscore,the validation will succeed

Thanks for explaining. 
The regex is correct.


- Matt


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


On Jan. 11, 2016, 9:26 p.m., yao lei wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42155/
> -----------------------------------------------------------
> 
> (Updated Jan. 11, 2016, 9:26 p.m.)
> 
> 
> Review request for Ambari, Andriy Babiichuk, Andrii Tkach, Jaimin Jetly, Matt, and Oleg
Nechiporenko.
> 
> 
> Bugs: AMBARI-14605
>     https://issues.apache.org/jira/browse/AMBARI-14605
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> HDFS now support four kinds of heterogeneous storage types: ARCHIVE, DISK, SSD and RAM_DISK.
> If we configure dfs.datanode.data.dir as [RAM_DISK]/anydir,it fails to validate.
> please see https://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-hdfs/ArchivalStorage.html#Configuration
> A
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/utils/validator.js 82c9f47 
>   ambari-web/test/utils/validator_test.js 5591aee 
> 
> Diff: https://reviews.apache.org/r/42155/diff/
> 
> 
> Testing
> -------
> 
> Manually Tested.
> But Ambari Web Unit Test fails in trunk for other reasons described in https://builds.apache.org/job/Ambari-trunk-test-patch/4848//artifact/patch-work/testrun_ambari-web.txt
> 
> 
> Thanks,
> 
> yao lei
> 
>


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