Hello Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3619
to look at the new patch set (#13).
Change subject: Doxygen for C++ client API
..
Doxygen for C++
Kudu Jenkins has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 13:
Build Started http://104.196.14.100/job/kudu-gerrit/2614/
--
To view, visit http://gerrit.cloudera.org:8080/3619
To
Alexey Serbin has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 12:
(42 comments)
http://gerrit.cloudera.org:8080/#/c/3619/12/src/kudu/client/client.h
File src/kudu/client/client.h:
Dan Burkert has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 12:
OK that's all my feedback. Thanks again for taking this on!
--
To view, visit http://gerrit.cloudera.org:8080/3619
Dan Burkert has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 12:
(19 comments)
http://gerrit.cloudera.org:8080/#/c/3619/12/src/kudu/client/client.h
File src/kudu/client/client.h:
Kudu Jenkins has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 12:
Build Started http://104.196.14.100/job/kudu-gerrit/2566/
--
To view, visit http://gerrit.cloudera.org:8080/3619
To
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3619
to look at the new patch set (#12).
Change subject: Doxygen for C++ client API
..
Doxygen for C++ client API
If
Alexey Serbin has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 9:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/3619/9/src/kudu/client/client.h
File src/kudu/client/client.h:
>
Adar Dembo has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 11:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/3619/9/src/kudu/client/client.h
File src/kudu/client/client.h:
> 1.
Kudu Jenkins has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 11:
Build Started http://104.196.14.100/job/kudu-gerrit/2551/
--
To view, visit http://gerrit.cloudera.org:8080/3619
To
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3619
to look at the new patch set (#11).
Change subject: Doxygen for C++ client API
..
Doxygen for C++ client API
If
Kudu Jenkins has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 10:
Build Started http://104.196.14.100/job/kudu-gerrit/2550/
--
To view, visit http://gerrit.cloudera.org:8080/3619
To
Alexey Serbin has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 9:
(64 comments)
http://gerrit.cloudera.org:8080/#/c/3619/9/src/kudu/client/client.h
File src/kudu/client/client.h:
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3619
to look at the new patch set (#10).
Change subject: doxygen for C++ client API
..
doxygen for C++ client API
If
Alexey Serbin has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 9:
(1 comment)
> One thing we should add to this (it can be a follow-up commit).
>
> It needs to be integrated into
Adar Dembo has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 9:
(65 comments)
http://gerrit.cloudera.org:8080/#/c/3619/9/src/kudu/client/client.h
File src/kudu/client/client.h:
High
Mike Percy has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 9:
One thing we should add to this (it can be a follow-up commit).
It needs to be integrated into make_site.sh and the web
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3619
to look at the new patch set (#9).
Change subject: doxygen for C++ client API
..
doxygen for C++ client API
If
Alexey Serbin has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 8:
(2 comments)
Will send an update shortly.
http://gerrit.cloudera.org:8080/#/c/3619/8/CMakeLists.txt
File
Kudu Jenkins has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 9:
Build Started http://104.196.14.100/job/kudu-gerrit/2509/
--
To view, visit http://gerrit.cloudera.org:8080/3619
To
Adar Dembo has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 8:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/3619/8/CMakeLists.txt
File CMakeLists.txt:
Line 989:
Alexey Serbin has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 8:
(4 comments)
Please see the responses in-line. Once the questions are cleared, I'll post
the next version.
Alexey Serbin has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 8:
(6 comments)
Thank you for review!
http://gerrit.cloudera.org:8080/#/c/3619/8/CMakeLists.txt
File CMakeLists.txt:
Adar Dembo has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 8:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/3619/8/CMakeLists.txt
File CMakeLists.txt:
Line 983:
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3619
to look at the new patch set (#8).
Change subject: doxygen for C++ client API
..
doxygen for C++ client API
If
Kudu Jenkins has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 8:
Build Started http://104.196.14.100/job/kudu-gerrit/2504/
--
To view, visit http://gerrit.cloudera.org:8080/3619
To
Kudu Jenkins has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 7:
Build Started http://104.196.14.100/job/kudu-gerrit/2503/
--
To view, visit http://gerrit.cloudera.org:8080/3619
To
27 matches
Mail list logo