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++ clien
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:
L
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
T
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:
Lin
Dan Burkert has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 12:
(23 comments)
I'm still working on this so don't push quite yet.
http://gerrit.cloudera.org:8080/#/c/3619/12/src/kudu
Adar Dembo has posted comments on this change.
Change subject: Doxygen for C++ client API
..
Patch Set 12: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/3619
To unsubscribe, visit http://gerrit.cloudera.org:8
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 do
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:
> 1.
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 do
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:
PS9
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 do
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 m
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 l
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 dox
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 CMakeLists.
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 u
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: add_custom_ta
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.
Thanks!
Adar Dembo has posted comments on this change.
Change subject: doxygen for C++ client API
..
Patch Set 8:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/3619/8/CMakeLists.txt
File CMakeLists.txt:
Line 983: set(DOXY_SUBD
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:
L
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: set(DOXY_SUBD
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 dox
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 u
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 u
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 (#7).
Change subject: doxygen for C++ client API
..
doxygen for C++ client API
To gen
31 matches
Mail list logo