[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16625 ) Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. KUDU-2612: C++ client sets txn_id in WriteRequestPB Thi

[kudu-CR] Fix order of clearing openssl error and printing it

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16631 ) Change subject: Fix order of clearing openssl error and printing it .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/

[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/16625 ) Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.

[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16625 ) Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloud

[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16625 ) Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Alexey Serbin (Code Review)
Hello Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16625 to look at the new patch set (#2). Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. K

[kudu-CR] [common] introduce TxnId wrapper

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16624 ) Change subject: [common] introduce TxnId wrapper .. [common] introduce TxnId wrapper TxnId is a wrapper around int64_t with tr

[kudu-CR] [partitioning] KUDU-2671: Support for range specific HashSchemas.

2020-10-22 Thread Mahesh Reddy (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Grant Henke, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16596 to look at the new patch set (#2). Change subject: [partitioning] KUDU-2671: Support for range specific HashSchemas. .

[kudu-CR] [partitioning] KUDU-2671: Support for range specific HashSchemas.

2020-10-22 Thread Mahesh Reddy (Code Review)
Mahesh Reddy has posted comments on this change. ( http://gerrit.cloudera.org:8080/16596 ) Change subject: [partitioning] KUDU-2671: Support for range specific HashSchemas. .. Patch Set 1: (10 comments) http://gerrit.clouder

[kudu-CR] Fix order of clearing openssl error and printing it

2020-10-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16631 ) Change subject: Fix order of clearing openssl error and printing it .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/16

[kudu-CR] Add locking to TLSContext & handshake

2020-10-22 Thread Attila Bukor (Code Review)
Attila Bukor has abandoned this change. ( http://gerrit.cloudera.org:8080/16635 ) Change subject: Add locking to TLSContext & handshake .. Abandoned wanted to push only the first commit -- To view, visit http://gerrit.cloudera

[kudu-CR] fips changes

2020-10-22 Thread Attila Bukor (Code Review)
Attila Bukor has abandoned this change. ( http://gerrit.cloudera.org:8080/16633 ) Change subject: fips changes .. Abandoned wanted to push only the first commit -- To view, visit http://gerrit.cloudera.org:8080/16633 To unsubs

[kudu-CR] add lock before verifying signature

2020-10-22 Thread Attila Bukor (Code Review)
Attila Bukor has abandoned this change. ( http://gerrit.cloudera.org:8080/16634 ) Change subject: add lock before verifying signature .. Abandoned wanted to push only the first commit -- To view, visit http://gerrit.cloudera.o

[kudu-CR] Fix order of clearing openssl error and printing it

2020-10-22 Thread Attila Bukor (Code Review)
Hello Alexey Serbin, Andrew Wong, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16631 to review the following change. Change subject: Fix order of clearing openssl error and printing it ..

[kudu-CR] Add FIPS mode

2020-10-22 Thread Attila Bukor (Code Review)
Attila Bukor has abandoned this change. ( http://gerrit.cloudera.org:8080/16632 ) Change subject: Add FIPS mode .. Abandoned wanted to push only the first commit -- To view, visit http://gerrit.cloudera.org:8080/16632 To unsub

[kudu-CR] add lock before verifying signature

2020-10-22 Thread Attila Bukor (Code Review)
Hello Alexey Serbin, Andrew Wong, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16634 to review the following change. Change subject: add lock before verifying signature .. ad

[kudu-CR] Add locking to TLSContext & handshake

2020-10-22 Thread Attila Bukor (Code Review)
Hello Alexey Serbin, Andrew Wong, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16635 to review the following change. Change subject: Add locking to TLSContext & handshake ..

[kudu-CR] Add FIPS mode

2020-10-22 Thread Attila Bukor (Code Review)
Hello Alexey Serbin, Andrew Wong, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16632 to review the following change. Change subject: Add FIPS mode .. Add FIPS mode Change-Id

[kudu-CR] fips changes

2020-10-22 Thread Attila Bukor (Code Review)
Hello Alexey Serbin, Andrew Wong, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16633 to review the following change. Change subject: fips changes .. fips changes Change-Id:

[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16625 ) Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. Patch Set 1: Code-Review+2 (2 comments) http://gerrit.cloudera.o

[kudu-CR] [backup] Fix Spark 2 and Scala 2.11 compilation

2020-10-22 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16629 ) Change subject: [backup] Fix Spark 2 and Scala 2.11 compilation .. [backup] Fix Spark 2 and Scala 2.11 compilation When compilin

[kudu-CR] [backup] Fix Spark 2 and Scala 2.11 compilation

2020-10-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16629 ) Change subject: [backup] Fix Spark 2 and Scala 2.11 compilation .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.

[kudu-CR](gh-pages) site: add an ecosystem page

2020-10-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 ) Change subject: site: add an ecosystem page .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/16615 To unsub

[kudu-CR](gh-pages) site: add an ecosystem page

2020-10-22 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16615 ) Change subject: site: add an ecosystem page .. site: add an ecosystem page I mostly pasted blurbs I found on each components' we

[kudu-CR] [common] introduce TxnId wrapper

2020-10-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16624 ) Change subject: [common] introduce TxnId wrapper .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16624

[kudu-CR](gh-pages) site: add an ecosystem page

2020-10-22 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 ) Change subject: site: add an ecosystem page .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16615 To u

[kudu-CR](gh-pages) site: add an ecosystem page

2020-10-22 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 ) Change subject: site: add an ecosystem page .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/16615 To un

[kudu-CR] [backup] Fix Spark 2 and Scala 2.11 compilation

2020-10-22 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16629 Change subject: [backup] Fix Spark 2 and Scala 2.11 compilation .. [backup] Fix Spark 2 and Scala 2.11 compilation When compiling with Sp

[kudu-CR] [tool] fix inconsistent definition among tool action common header and src

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16618 ) Change subject: [tool] fix inconsistent definition among tool_action_common header and src .. Patch Set 6: Code-Review+2 Looks good t

[kudu-CR] [tool] fix inconsistent definition among tool action common header and src

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16618 ) Change subject: [tool] fix inconsistent definition among tool_action_common header and src .. [tool] fix inconsistent definiti

[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/16625 ) Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] KUDU-2612: C++ client sets txn id in WriteRequestPB

2020-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16625 ) Change subject: KUDU-2612: C++ client sets txn_id in WriteRequestPB .. Patch Set 1: Verified+1 unrelated test failure already reported

[kudu-CR] [tool] fix inconsistent definition among tool action common header and src

2020-10-22 Thread Anonymous Coward (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16618 to look at the new patch set (#6). Change subject: [tool] fix inconsistent definition among tool_action_common header and src ..

[kudu-CR] [tool] fix inconsistent definition among tool action common header and src

2020-10-22 Thread Anonymous Coward (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16618 to look at the new patch set (#5). Change subject: [tool] fix inconsistent definition among tool_action_common header and src ..

[kudu-CR] [tool] fix inconsistent definition among tool action common header and src

2020-10-22 Thread Anonymous Coward (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16618 to look at the new patch set (#4). Change subject: [tool] fix inconsistent definition among tool_action_common header and src ..

[kudu-CR] [tool] fix inconsistent definition among tool action common header and src

2020-10-22 Thread Anonymous Coward (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16618 to look at the new patch set (#3). Change subject: [tool] fix inconsistent definition among tool_action_common header and src ..

[kudu-CR] [tool] fix inconsistent definition among tool action common header and src

2020-10-22 Thread Anonymous Coward (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16618 to look at the new patch set (#2). Change subject: [tool] fix inconsistent definition among tool_action_common header and src ..