mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Klaus Ma <klaus1982...@gmail.com>
Subject Re: Review Request 49223: Enhance value parsing.
Date Tue, 05 Jul 2016 06:56:16 GMT


> On July 4, 2016, 7:55 p.m., Guangya Liu wrote:
> > src/common/values.cpp, line 673
> > <https://reviews.apache.org/r/49223/diff/4/?file=1436140#file1436140line673>
> >
> >     Sorry, I should ask this question in previous patch. Same as above, can you
please show more comments for what do you want to check here? It would be great if you can
make the comment easy to understand.
> 
> Klaus Ma wrote:
>     Only [0-9a-z./] is available for Text in document.
> 
> Guangya Liu wrote:
>     OK, then what about "Check `text resource` in the format of `[a-zA-Z0-9_/.-]`.",
ditto for above.

1, Text is not resource; 2. I'd like to depdent on document; otherwise, we need to align TODO
and doc.


- Klaus


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


On July 4, 2016, 6:33 p.m., Klaus Ma wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49223/
> -----------------------------------------------------------
> 
> (Updated July 4, 2016, 6:33 p.m.)
> 
> 
> Review request for mesos and Benjamin Mahler.
> 
> 
> Bugs: MESOS-5739
>     https://issues.apache.org/jira/browse/MESOS-5739
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Enhance value parsing.
> 
> 
> Diffs
> -----
> 
>   src/common/values.cpp 587cb68551d438621e215953e89818b623b7f71b 
>   src/tests/values_tests.cpp 929861549e3155c33966896f817f9bf9e6d14354 
> 
> Diff: https://reviews.apache.org/r/49223/diff/
> 
> 
> Testing
> -------
> 
> make && make check
> 
> 
> Thanks,
> 
> Klaus Ma
> 
>


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