You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Benjamin Hindman <be...@berkeley.edu> on 2012/08/08 22:20:34 UTC

Review Request: Added a 'files' abstraction for browsing and reading files from HTTP endpoints.

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

Review request for mesos and Ben Mahler.


Description
-------

See summary.


Diffs
-----

  src/Makefile.am cc3480c 
  src/files/files.hpp PRE-CREATION 
  src/files/files.cpp PRE-CREATION 
  src/master/http.cpp 6fb2a7d 
  third_party/libprocess/include/stout/os.hpp e9078cc 
  third_party/libprocess/include/stout/strings.hpp 557a6a8 

Diff: https://reviews.apache.org/r/6477/diff/


Testing
-------

make check


Thanks,

Benjamin Hindman


Re: Review Request: Added a 'files' abstraction for browsing and reading files from HTTP endpoints.

Posted by Ben Mahler <be...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6477/#review10044
-----------------------------------------------------------

Ship it!


Ship It!

- Ben Mahler


On Aug. 8, 2012, 10:48 p.m., Benjamin Hindman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6477/
> -----------------------------------------------------------
> 
> (Updated Aug. 8, 2012, 10:48 p.m.)
> 
> 
> Review request for mesos and Ben Mahler.
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   src/Makefile.am cc3480c 
>   src/files/files.hpp PRE-CREATION 
>   src/files/files.cpp PRE-CREATION 
>   src/master/http.hpp e390034 
>   src/master/http.cpp 6fb2a7d 
>   src/master/master.hpp 30e3f75 
>   src/master/master.cpp b174f7f 
>   src/slave/http.cpp 0780e5f 
>   src/slave/slave.hpp 56b4ee2 
>   src/slave/slave.cpp d2cd62f 
>   src/webui/master/static/app.js c2570a7 
>   src/webui/master/static/controllers.js 5e6be19 
>   src/webui/master/static/home.html 09892cf 
>   src/webui/master/static/jquery.pailer.js e7120b1 
>   src/webui/master/static/log.html 5bee260 
>   src/webui/master/static/pailer.html PRE-CREATION 
>   src/webui/master/static/slave.html PRE-CREATION 
>   src/webui/master/static/slaves.html 7932ccd 
>   third_party/libprocess/include/stout/os.hpp e9078cc 
>   third_party/libprocess/include/stout/strings.hpp 557a6a8 
> 
> Diff: https://reviews.apache.org/r/6477/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Benjamin Hindman
> 
>


Re: Review Request: Added a 'files' abstraction for browsing and reading files from HTTP endpoints.

Posted by Benjamin Hindman <be...@berkeley.edu>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6477/
-----------------------------------------------------------

(Updated Aug. 8, 2012, 10:48 p.m.)


Review request for mesos and Ben Mahler.


Changes
-------

Compilation fixes.


Description
-------

See summary.


Diffs (updated)
-----

  src/Makefile.am cc3480c 
  src/files/files.hpp PRE-CREATION 
  src/files/files.cpp PRE-CREATION 
  src/master/http.hpp e390034 
  src/master/http.cpp 6fb2a7d 
  src/master/master.hpp 30e3f75 
  src/master/master.cpp b174f7f 
  src/slave/http.cpp 0780e5f 
  src/slave/slave.hpp 56b4ee2 
  src/slave/slave.cpp d2cd62f 
  src/webui/master/static/app.js c2570a7 
  src/webui/master/static/controllers.js 5e6be19 
  src/webui/master/static/home.html 09892cf 
  src/webui/master/static/jquery.pailer.js e7120b1 
  src/webui/master/static/log.html 5bee260 
  src/webui/master/static/pailer.html PRE-CREATION 
  src/webui/master/static/slave.html PRE-CREATION 
  src/webui/master/static/slaves.html 7932ccd 
  third_party/libprocess/include/stout/os.hpp e9078cc 
  third_party/libprocess/include/stout/strings.hpp 557a6a8 

Diff: https://reviews.apache.org/r/6477/diff/


Testing
-------

make check


Thanks,

Benjamin Hindman


Re: Review Request: Added a 'files' abstraction for browsing and reading files from HTTP endpoints.

Posted by Benjamin Hindman <be...@berkeley.edu>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6477/
-----------------------------------------------------------

(Updated Aug. 8, 2012, 9:59 p.m.)


Review request for mesos and Ben Mahler.


Changes
-------

Updated to include JSONP additions and hook up to the webui.


Description
-------

See summary.


Diffs (updated)
-----

  src/Makefile.am cc3480c 
  src/files/files.hpp PRE-CREATION 
  src/files/files.cpp PRE-CREATION 
  src/master/http.hpp e390034 
  src/master/http.cpp 6fb2a7d 
  src/master/master.hpp 30e3f75 
  src/master/master.cpp b174f7f 
  src/slave/http.cpp 0780e5f 
  src/slave/slave.hpp 56b4ee2 
  src/slave/slave.cpp d2cd62f 
  src/webui/master/static/app.js c2570a7 
  src/webui/master/static/controllers.js 5e6be19 
  src/webui/master/static/home.html 09892cf 
  src/webui/master/static/jquery.pailer.js e7120b1 
  src/webui/master/static/log.html 5bee260 
  src/webui/master/static/pailer.html PRE-CREATION 
  src/webui/master/static/slave.html PRE-CREATION 
  src/webui/master/static/slaves.html 7932ccd 
  third_party/libprocess/include/stout/os.hpp e9078cc 
  third_party/libprocess/include/stout/strings.hpp 557a6a8 

Diff: https://reviews.apache.org/r/6477/diff/


Testing
-------

make check


Thanks,

Benjamin Hindman