directly from the
specified URI, and the with-cache path, where it copies it from the cache. In
both cases, we verify that the file is saved to the sandbox directory with the
name specified by the "filename" field in the CommandInfo.URI proto.
Thanks,
Michael Browning
saved to the sandbox directory with the
name specified by the "filename" field in the CommandInfo.URI proto.
Thanks,
Michael Browning
---------
On March 18, 2016, 11:30 p.m., Michael Browning wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45046/
>
eld in the CommandInfo.URI proto.
Thanks,
Michael Browning
-- should I also create an issue for that?
- Michael
-------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45046/#review124456
---
directly from the
specified URI, and the with-cache path, where it copies it from the cache. In
both cases, we verify that the file is saved to the sandbox directory with the
name specified by the "filename" field in the CommandInfo.URI proto.
Thanks,
Michael Browning
, we verify that the file is saved to the sandbox directory with the
name specified by the "filename" field in the CommandInfo.URI proto.
Thanks,
Michael Browning
performed.
Thanks,
Michael Browning
_tests.cpp,
CachedCustomFilenameWithSubdirectory tests that the same behavior holds when
the URI is fetched from the cache.
Thanks,
Michael Browning
jected. In fetcher_cache_tests.cpp,
CachedCustomFilenameWithSubdirectory tests that the same behavior holds when
the URI is fetched from the cache.
Thanks,
Michael Browning
ps://reviews.apache.org/r/46168/diff/1/?file=1343102#file1343102line150>
> >
> > What does this check against?
> >
> > Path(filename).basename() never returns an Error().
Ah, got it confused with Fetcher::basename, which can return an Error.
-
sandbox, and
AbsoluteCustomSubdirectoryFails tests that a custom filename with an absolute
path is rejected. In fetcher_cache_tests.cpp,
CachedCustomFilenameWithSubdirectory tests that the same behavior holds when
the URI is fetched from the cache.
Thanks,
Michael Browning
ndbox, and
AbsoluteCustomSubdirectoryFails tests that a custom filename with an absolute
path is rejected. In fetcher_cache_tests.cpp,
CachedCustomFilenameWithSubdirectory tests that the same behavior holds when
the URI is fetched from the cache.
Thanks,
Michael Browning
fetcher_cache_tests.cpp,
CachedCustomFilenameWithSubdirectory tests that the same behavior holds when
the URI is fetched from the cache.
Thanks,
Michael Browning
jected. In fetcher_cache_tests.cpp,
CachedCustomFilenameWithSubdirectory tests that the same behavior holds when
the URI is fetched from the cache.
Thanks,
Michael Browning
---
This change involves changing existing macro invocations in testing code, so no
additional testing was performed.
Thanks,
Michael Browning
://reviews.apache.org/r/43587/diff/
Testing
---
This change involves changing existing macro invocations in testing code, so no
additional testing was performed.
Thanks,
Michael Browning
/slave/metrics.cpp 86eb8db644227cb593e52305bfbd05444bb87a6e
src/slave/slave.hpp 57b18882e30e44dcc40449b0e3be8ee970c45bc8
Diff: https://reviews.apache.org/r/46717/diff/
Testing
---
Thanks,
Michael Browning
18 matches
Mail list logo