[kudu-CR](gh-pages) Update the releases index page for release 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17006 )

Change subject: Update the releases index page for release 1.14.0
..


Patch Set 3: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/17006
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Iac6addef0ad29d5780860631d83ce14b4a9fbf5d
Gerrit-Change-Number: 17006
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Thu, 28 Jan 2021 22:14:43 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) Update the releases index page for release 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Attila Bukor, Hao Hao,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/17006

to look at the new patch set (#3).

Change subject: Update the releases index page for release 1.14.0
..

Update the releases index page for release 1.14.0

Change-Id: Iac6addef0ad29d5780860631d83ce14b4a9fbf5d
---
M releases/index.md
1 file changed, 5 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/17006/3
--
To view, visit http://gerrit.cloudera.org:8080/17006
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iac6addef0ad29d5780860631d83ce14b4a9fbf5d
Gerrit-Change-Number: 17006
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Hao Hao 


[kudu-CR](gh-pages) Update the releases index page for release 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Attila Bukor, Hao Hao,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/17006

to look at the new patch set (#2).

Change subject: Update the releases index page for release 1.14.0
..

Update the releases index page for release 1.14.0

Change-Id: Iac6addef0ad29d5780860631d83ce14b4a9fbf5d
---
M releases/index.md
1 file changed, 5 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/17006/2
--
To view, visit http://gerrit.cloudera.org:8080/17006
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iac6addef0ad29d5780860631d83ce14b4a9fbf5d
Gerrit-Change-Number: 17006
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Hao Hao 


[kudu-CR](gh-pages) Update the releases index page for release 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17006


Change subject: Update the releases index page for release 1.14.0
..

Update the releases index page for release 1.14.0

Change-Id: Iac6addef0ad29d5780860631d83ce14b4a9fbf5d
---
M releases/index.md
1 file changed, 5 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/17006/1
--
To view, visit http://gerrit.cloudera.org:8080/17006
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iac6addef0ad29d5780860631d83ce14b4a9fbf5d
Gerrit-Change-Number: 17006
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR](gh-pages) Update website for 1.14.0 release

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16990 )

Change subject: Update website for 1.14.0 release
..

Update website for 1.14.0 release

Change-Id: Ia97cc8456a8ad0a81bb9732c7576641f41e165aa
Reviewed-on: http://gerrit.cloudera.org:8080/16990
Reviewed-by: Hao Hao 
Reviewed-by: Alexey Serbin 
Tested-by: Grant Henke 
---
M apidocs
M cpp-client-api
M docs
A releases/1.14.0/apidocs/allclasses-frame.html
A releases/1.14.0/apidocs/allclasses-noframe.html
A releases/1.14.0/apidocs/constant-values.html
A releases/1.14.0/apidocs/deprecated-list.html
A releases/1.14.0/apidocs/help-doc.html
A releases/1.14.0/apidocs/index-all.html
A releases/1.14.0/apidocs/index.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.ColumnSchemaBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.CompressionAlgorithm.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.Encoding.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.html
A 
releases/1.14.0/apidocs/org/apache/kudu/ColumnTypeAttributes.ColumnTypeAttributesBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnTypeAttributes.html
A releases/1.14.0/apidocs/org/apache/kudu/Schema.html
A releases/1.14.0/apidocs/org/apache/kudu/Type.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AbstractKuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AlterTableOptions.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AlterTableResponse.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduClient.AsyncKuduClientBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduClient.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.AsyncKuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.ReadMode.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.RowDataFormat.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduSession.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ColumnRangePredicate.html
A releases/1.14.0/apidocs/org/apache/kudu/client/CreateTableOptions.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Delete.html
A releases/1.14.0/apidocs/org/apache/kudu/client/DeleteIgnore.html
A releases/1.14.0/apidocs/org/apache/kudu/client/DeleteTableResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ExternalConsistencyMode.html
A releases/1.14.0/apidocs/org/apache/kudu/client/HasFailedRpcException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Insert.html
A releases/1.14.0/apidocs/org/apache/kudu/client/InsertIgnore.html
A releases/1.14.0/apidocs/org/apache/kudu/client/IsAlterTableDoneResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/IsCreateTableDoneResponse.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduClient.KuduClientBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduClient.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduException.OriginalException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduException.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduPartitioner.KuduPartitionerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPartitioner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPredicate.ComparisonOp.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPredicate.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanToken.KuduScanTokenBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanToken.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanner.KuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScannerIterator.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduSession.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTable.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTableStatistics.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTransaction.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ListTablesResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ListTabletServersResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/LocatedTablet.Replica.html
A releases/1.14.0/apidocs/org/apache/kudu/client/LocatedTablet.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Operation.html
A releases/1.14.0/apidocs/org/apache/kudu/client/OperationResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/PartialRow.html
A releases/1.14.0/apidocs/org/apache/kudu/client/PleaseThrottleException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/RangePartitionBound.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ReplicaSelection.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ResourceMetrics.html
A 

[kudu-CR](gh-pages) [blog] Add post announcing Kudu 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16998 )

Change subject: [blog] Add post announcing Kudu 1.14.0
..

[blog] Add post announcing Kudu 1.14.0

Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Reviewed-on: http://gerrit.cloudera.org:8080/16998
Reviewed-by: Alexey Serbin 
Reviewed-by: Attila Bukor 
Tested-by: Grant Henke 
---
A _posts/2021–01-28-apache-kudu-1-14-0-release.md
1 file changed, 76 insertions(+), 0 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Attila Bukor: Looks good to me, approved
  Grant Henke: Verified

--
To view, visit http://gerrit.cloudera.org:8080/16998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: merged
Gerrit-Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Gerrit-Change-Number: 16998
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Hao Hao 


[kudu-CR](gh-pages) Update website for 1.14.0 release

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16990 )

Change subject: Update website for 1.14.0 release
..


Patch Set 2: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/16990
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia97cc8456a8ad0a81bb9732c7576641f41e165aa
Gerrit-Change-Number: 16990
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Thu, 28 Jan 2021 22:00:48 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) [blog] Add post announcing Kudu 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16998 )

Change subject: [blog] Add post announcing Kudu 1.14.0
..


Patch Set 3: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/16998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Gerrit-Change-Number: 16998
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Thu, 28 Jan 2021 20:38:55 +
Gerrit-HasComments: No


[kudu-CR](branch-1.14.x) KUDU-3239: [build] Disable errorprone on Java 11

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16986 )

Change subject: KUDU-3239: [build] Disable errorprone on Java 11
..

KUDU-3239: [build] Disable errorprone on Java 11

It turns out that errorprone is not working with Java 11. This patch
disables it until we can upgrade/update the errorprone plugin to
work on with Java 11.

Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Reviewed-on: http://gerrit.cloudera.org:8080/16983
Reviewed-by: Bankim Bhavsar 
Reviewed-by: Hao Hao 
Reviewed-by: Alexey Serbin 
Tested-by: Kudu Jenkins
(cherry picked from commit 922ab9e7d46f683a5e04b4e36b1af9b95e96e025)
Reviewed-on: http://gerrit.cloudera.org:8080/16986
Tested-by: Grant Henke 
---
M java/gradle/quality.gradle
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Bankim Bhavsar: Looks good to me, approved
  Alexey Serbin: Looks good to me, approved
  Grant Henke: Verified

--
To view, visit http://gerrit.cloudera.org:8080/16986
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Gerrit-Change-Number: 16986
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) KUDU-3239: [build] Disable errorprone on Java 11

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change.

Change subject: KUDU-3239: [build] Disable errorprone on Java 11
..


Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/16986
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Gerrit-Change-Number: 16986
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) KUDU-3239: [build] Disable errorprone on Java 11

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16986 )

Change subject: KUDU-3239: [build] Disable errorprone on Java 11
..


Patch Set 1: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/16986
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Gerrit-Change-Number: 16986
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 28 Jan 2021 19:39:56 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) [blog] Add post announcing Kudu 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16998 )

Change subject: [blog] Add post announcing Kudu 1.14.0
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16998/2/_posts/2021%E2%80%9301-28-apache-kudu-1-14-0-release.md
File _posts/2021–01-28-apache-kudu-1-14-0-release.md:

http://gerrit.cloudera.org:8080/#/c/16998/2/_posts/2021%E2%80%9301-28-apache-kudu-1-14-0-release.md@36
PS2, Line 36: C++ client by setting the KUDU_REQUIRE_FIPS_MODE
> I am okay to remove it.
Done



--
To view, visit http://gerrit.cloudera.org:8080/16998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Gerrit-Change-Number: 16998
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Thu, 28 Jan 2021 19:39:14 +
Gerrit-HasComments: Yes


[kudu-CR](gh-pages) [blog] Add post announcing Kudu 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Attila Bukor, Andrew Wong, Greg Solovyev, Hao Hao, Bankim 
Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16998

to look at the new patch set (#3).

Change subject: [blog] Add post announcing Kudu 1.14.0
..

[blog] Add post announcing Kudu 1.14.0

Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
---
A _posts/2021–01-28-apache-kudu-1-14-0-release.md
1 file changed, 76 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/98/16998/3
--
To view, visit http://gerrit.cloudera.org:8080/16998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Gerrit-Change-Number: 16998
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Hao Hao 


[kudu-CR](gh-pages) Update website for 1.14.0 release

2021-01-28 Thread Grant Henke (Code Review)
Hello Attila Bukor, Andrew Wong, Hao Hao,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16990

to look at the new patch set (#2).

Change subject: Update website for 1.14.0 release
..

Update website for 1.14.0 release

Change-Id: Ia97cc8456a8ad0a81bb9732c7576641f41e165aa
---
M apidocs
M cpp-client-api
M docs
A releases/1.14.0/apidocs/allclasses-frame.html
A releases/1.14.0/apidocs/allclasses-noframe.html
A releases/1.14.0/apidocs/constant-values.html
A releases/1.14.0/apidocs/deprecated-list.html
A releases/1.14.0/apidocs/help-doc.html
A releases/1.14.0/apidocs/index-all.html
A releases/1.14.0/apidocs/index.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.ColumnSchemaBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.CompressionAlgorithm.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.Encoding.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.html
A 
releases/1.14.0/apidocs/org/apache/kudu/ColumnTypeAttributes.ColumnTypeAttributesBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnTypeAttributes.html
A releases/1.14.0/apidocs/org/apache/kudu/Schema.html
A releases/1.14.0/apidocs/org/apache/kudu/Type.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AbstractKuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AlterTableOptions.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AlterTableResponse.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduClient.AsyncKuduClientBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduClient.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.AsyncKuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.ReadMode.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.RowDataFormat.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduSession.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ColumnRangePredicate.html
A releases/1.14.0/apidocs/org/apache/kudu/client/CreateTableOptions.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Delete.html
A releases/1.14.0/apidocs/org/apache/kudu/client/DeleteIgnore.html
A releases/1.14.0/apidocs/org/apache/kudu/client/DeleteTableResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ExternalConsistencyMode.html
A releases/1.14.0/apidocs/org/apache/kudu/client/HasFailedRpcException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Insert.html
A releases/1.14.0/apidocs/org/apache/kudu/client/InsertIgnore.html
A releases/1.14.0/apidocs/org/apache/kudu/client/IsAlterTableDoneResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/IsCreateTableDoneResponse.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduClient.KuduClientBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduClient.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduException.OriginalException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduException.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduPartitioner.KuduPartitionerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPartitioner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPredicate.ComparisonOp.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPredicate.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanToken.KuduScanTokenBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanToken.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanner.KuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScannerIterator.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduSession.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTable.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTableStatistics.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTransaction.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ListTablesResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ListTabletServersResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/LocatedTablet.Replica.html
A releases/1.14.0/apidocs/org/apache/kudu/client/LocatedTablet.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Operation.html
A releases/1.14.0/apidocs/org/apache/kudu/client/OperationResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/PartialRow.html
A releases/1.14.0/apidocs/org/apache/kudu/client/PleaseThrottleException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/RangePartitionBound.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ReplicaSelection.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ResourceMetrics.html
A 

[kudu-CR](gh-pages) [blog] Add post announcing Kudu 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16998 )

Change subject: [blog] Add post announcing Kudu 1.14.0
..


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/16998/2/_posts/2021%E2%80%9301-28-apache-kudu-1-14-0-release.md
File _posts/2021–01-28-apache-kudu-1-14-0-release.md:

http://gerrit.cloudera.org:8080/#/c/16998/2/_posts/2021%E2%80%9301-28-apache-kudu-1-14-0-release.md@36
PS2, Line 36: C++ client by setting the KUDU_REQUIRE_FIPS_MODE
> When reading this, I thought 'What about the Java client?'.  From that pers
I am okay to remove it.


http://gerrit.cloudera.org:8080/#/c/16998/2/_posts/2021%E2%80%9301-28-apache-kudu-1-14-0-release.md@36
PS2, Line 36: C++ client by setting the KUDU_REQUIRE_FIPS_MODE
> One more point here.
Feel free to file a Jira to summarize your thoughts and suggestions on the 
feature for future releases. For now this is just communicating the existing 
state and copied directly release notes.

I will remove it from the post.



--
To view, visit http://gerrit.cloudera.org:8080/16998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Gerrit-Change-Number: 16998
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Thu, 28 Jan 2021 18:41:25 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.14.x) [docker] Update pip install

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17004 )

Change subject: [docker] Update pip install
..

[docker] Update pip install

It looks like Python has adjusted the get-pip.py URL to no longer
support Python 2.7. This patch updates the Docker scripts to
use the 2.7 specific URL and drop the old 2.6 speicific url now
that no supported images use Python 2.6.

Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Reviewed-on: http://gerrit.cloudera.org:8080/16984
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
(cherry picked from commit b5e7362e69a5d7a9c9e88fa144e0366a2b22a83a)
Reviewed-on: http://gerrit.cloudera.org:8080/17004
---
M docker/bootstrap-dev-env.sh
M docker/bootstrap-python-env.sh
2 files changed, 7 insertions(+), 8 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17004
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 17004
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) [build] Fix occasional Gradle build failures

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17001 )

Change subject: [build] Fix occasional Gradle build failures
..

[build] Fix occasional Gradle build failures

Occasionally, on a clean build environment often as a part of the
CI pipline, the C++ build will fail with cryptic often Protobuf
related failures. It turns out this is a result of multiple
./gradlew calls which depend on the kudu-proto module.
This change fixes the issue by defining a single CMake target
to build the kudu-proto jar.

Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Reviewed-on: http://gerrit.cloudera.org:8080/16968
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Andrew Wong 
(cherry picked from commit 202bcae7f12ca0d7ac43e0a07e57f3236c8f968d)
Reviewed-on: http://gerrit.cloudera.org:8080/17001
Tested-by: Grant Henke 
Reviewed-by: Attila Bukor 
---
M CMakeLists.txt
M src/kudu/hms/CMakeLists.txt
M src/kudu/subprocess/CMakeLists.txt
3 files changed, 13 insertions(+), 2 deletions(-)

Approvals:
  Grant Henke: Verified
  Alexey Serbin: Looks good to me, approved
  Attila Bukor: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17001
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 17001
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) [docs] Update versions in docs to 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17003 )

Change subject: [docs] Update versions in docs to 1.14.0
..

[docs] Update versions in docs to 1.14.0

Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
Reviewed-on: http://gerrit.cloudera.org:8080/17000
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
(cherry picked from commit a62bfa305cd0d421183247349255b4032c56e7a3)
Reviewed-on: http://gerrit.cloudera.org:8080/17003
Reviewed-by: Attila Bukor 
---
M docs/administration.adoc
M docs/developing.adoc
M docs/quickstart.adoc
3 files changed, 9 insertions(+), 9 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved
  Attila Bukor: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17003
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
Gerrit-Change-Number: 17003
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) [docs] Add the 1.13.0 release notes to the prior release notes

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17002 )

Change subject: [docs] Add the 1.13.0 release notes to the prior release notes
..

[docs] Add the 1.13.0 release notes to the prior release notes

Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
Reviewed-on: http://gerrit.cloudera.org:8080/16996
Tested-by: Kudu Jenkins
Reviewed-by: Attila Bukor 
(cherry picked from commit 43ec35847be54197fe0ca83123629a101088a396)
Reviewed-on: http://gerrit.cloudera.org:8080/17002
Reviewed-by: Alexey Serbin 
---
M docs/prior_release_notes.adoc
1 file changed, 197 insertions(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved
  Attila Bukor: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17002
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
Gerrit-Change-Number: 17002
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) [docker] Update pip install

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17004


Change subject: [docker] Update pip install
..

[docker] Update pip install

It looks like Python has adjusted the get-pip.py URL to no longer
support Python 2.7. This patch updates the Docker scripts to
use the 2.7 specific URL and drop the old 2.6 speicific url now
that no supported images use Python 2.6.

Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Reviewed-on: http://gerrit.cloudera.org:8080/16984
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
(cherry picked from commit b5e7362e69a5d7a9c9e88fa144e0366a2b22a83a)
---
M docker/bootstrap-dev-env.sh
M docker/bootstrap-python-env.sh
2 files changed, 7 insertions(+), 8 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/17004/1
--
To view, visit http://gerrit.cloudera.org:8080/17004
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 17004
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [docker] Update pip install

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16984 )

Change subject: [docker] Update pip install
..

[docker] Update pip install

It looks like Python has adjusted the get-pip.py URL to no longer
support Python 2.7. This patch updates the Docker scripts to
use the 2.7 specific URL and drop the old 2.6 speicific url now
that no supported images use Python 2.6.

Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Reviewed-on: http://gerrit.cloudera.org:8080/16984
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M docker/bootstrap-dev-env.sh
M docker/bootstrap-python-env.sh
2 files changed, 7 insertions(+), 8 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16984
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 16984
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) [build] Fix occasional Gradle build failures

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17001 )

Change subject: [build] Fix occasional Gradle build failures
..


Patch Set 1: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/17001
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 17001
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 28 Jan 2021 17:36:49 +
Gerrit-HasComments: No


[kudu-CR](branch-1.14.x) [build] Fix occasional Gradle build failures

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change.

Change subject: [build] Fix occasional Gradle build failures
..


Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/17001
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 17001
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) [docs] Update versions in docs to 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17003


Change subject: [docs] Update versions in docs to 1.14.0
..

[docs] Update versions in docs to 1.14.0

Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
Reviewed-on: http://gerrit.cloudera.org:8080/17000
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
(cherry picked from commit a62bfa305cd0d421183247349255b4032c56e7a3)
---
M docs/administration.adoc
M docs/developing.adoc
M docs/quickstart.adoc
3 files changed, 9 insertions(+), 9 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/03/17003/1
--
To view, visit http://gerrit.cloudera.org:8080/17003
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
Gerrit-Change-Number: 17003
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [docs] Update versions in docs to 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17000 )

Change subject: [docs] Update versions in docs to 1.14.0
..

[docs] Update versions in docs to 1.14.0

Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
Reviewed-on: http://gerrit.cloudera.org:8080/17000
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M docs/administration.adoc
M docs/developing.adoc
M docs/quickstart.adoc
3 files changed, 9 insertions(+), 9 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17000
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
Gerrit-Change-Number: 17000
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [examples] Update versions in examples to 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16999 )

Change subject: [examples] Update versions in examples to 1.14.0
..

[examples] Update versions in examples to 1.14.0

Change-Id: I0254bdd321c1313f1042e9ef5a9ad1fbbec42436
Reviewed-on: http://gerrit.cloudera.org:8080/16999
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M examples/java/collectl/README.adoc
M examples/java/collectl/pom.xml
M examples/java/insert-loadgen/pom.xml
M examples/java/java-example/pom.xml
M examples/quickstart/nifi/README.adoc
M examples/quickstart/spark/README.adoc
M examples/quickstart/ycsb/README.adoc
M examples/scala/sbt-int-test-example/README.adoc
M examples/scala/sbt-int-test-example/build.sbt
M examples/scala/spark-example/pom.xml
10 files changed, 15 insertions(+), 15 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16999
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I0254bdd321c1313f1042e9ef5a9ad1fbbec42436
Gerrit-Change-Number: 16999
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [java] address warnings from the SonarLint tool

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16969 )

Change subject: [java] address warnings from the SonarLint tool
..


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java
File 
java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java:

http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java@51
PS1, Line 51: int numRows) {
> Thank you for pointing at this.
That makes sense. It looks like this is okay to change. Thanks for looking into 
it.


http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/util/AsyncUtil.java
File java/kudu-client/src/main/java/org/apache/kudu/util/AsyncUtil.java:

http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/util/AsyncUtil.java@75
PS1, Line 75:   private AsyncUtil() {
> I put this in the very end of the file to match what we already have in the
I generally expect to see constructors, if there are any, at the top of the 
class. Ultimately this is a nit, and I am okay with it either way.

The google style guide isn't as concrete on this, but the oracle guide suggests 
that constructors should come before methods.
https://google.github.io/styleguide/javaguide.html#s3.4.2-ordering-class-contents
https://www.oracle.com/java/technologies/javase/codeconventions-fileorganization.html#1852



--
To view, visit http://gerrit.cloudera.org:8080/16969
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I306575b370fb69a54fdc24b9d770a99291828ef7
Gerrit-Change-Number: 16969
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Comment-Date: Thu, 28 Jan 2021 16:07:17 +
Gerrit-HasComments: Yes


[kudu-CR] [docs] Add the 1.14.0 release notes to the prior release notes

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16997 )

Change subject: [docs] Add the 1.14.0 release notes to the prior release notes
..


Patch Set 1:

I think we can manually adjust any publish from master if needed by excluding 
certain files being changed. We generally need to do that anyway.


--
To view, visit http://gerrit.cloudera.org:8080/16997
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If757e53b6c014f2c9f13988b746a758fa2b23cfa
Gerrit-Change-Number: 16997
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 28 Jan 2021 15:45:37 +
Gerrit-HasComments: No


[kudu-CR](branch-1.14.x) [docs] Add the 1.13.0 release notes to the prior release notes

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17002


Change subject: [docs] Add the 1.13.0 release notes to the prior release notes
..

[docs] Add the 1.13.0 release notes to the prior release notes

Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
Reviewed-on: http://gerrit.cloudera.org:8080/16996
Tested-by: Kudu Jenkins
Reviewed-by: Attila Bukor 
(cherry picked from commit 43ec35847be54197fe0ca83123629a101088a396)
---
M docs/prior_release_notes.adoc
1 file changed, 197 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/02/17002/1
--
To view, visit http://gerrit.cloudera.org:8080/17002
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
Gerrit-Change-Number: 17002
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR](branch-1.14.x) [build] Fix occasional Gradle build failures

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17001


Change subject: [build] Fix occasional Gradle build failures
..

[build] Fix occasional Gradle build failures

Occasionally, on a clean build environment often as a part of the
CI pipline, the C++ build will fail with cryptic often Protobuf
related failures. It turns out this is a result of multiple
./gradlew calls which depend on the kudu-proto module.
This change fixes the issue by defining a single CMake target
to build the kudu-proto jar.

Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Reviewed-on: http://gerrit.cloudera.org:8080/16968
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Andrew Wong 
(cherry picked from commit 202bcae7f12ca0d7ac43e0a07e57f3236c8f968d)
---
M CMakeLists.txt
M src/kudu/hms/CMakeLists.txt
M src/kudu/subprocess/CMakeLists.txt
3 files changed, 13 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/01/17001/1
--
To view, visit http://gerrit.cloudera.org:8080/17001
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 17001
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR](branch-1.14.x) Bump version to 1.14.1-SNAPSHOT

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16994 )

Change subject: Bump version to 1.14.1-SNAPSHOT
..

Bump version to 1.14.1-SNAPSHOT

Change-Id: Id894844feac9fd8763e67f1bc1097256c6146bdd
Reviewed-on: http://gerrit.cloudera.org:8080/16994
Reviewed-by: Attila Bukor 
Tested-by: Grant Henke 
---
M version.txt
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Attila Bukor: Looks good to me, approved
  Grant Henke: Verified

--
To view, visit http://gerrit.cloudera.org:8080/16994
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Id894844feac9fd8763e67f1bc1097256c6146bdd
Gerrit-Change-Number: 16994
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) Bump version to 1.14.1-SNAPSHOT

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16994 )

Change subject: Bump version to 1.14.1-SNAPSHOT
..


Patch Set 1: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/16994
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Id894844feac9fd8763e67f1bc1097256c6146bdd
Gerrit-Change-Number: 16994
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 28 Jan 2021 15:38:33 +
Gerrit-HasComments: No


[kudu-CR](branch-1.14.x) Bump version to 1.14.1-SNAPSHOT

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change.

Change subject: Bump version to 1.14.1-SNAPSHOT
..


Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/16994
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Id894844feac9fd8763e67f1bc1097256c6146bdd
Gerrit-Change-Number: 16994
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [docker] Update pip install

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16984 )

Change subject: [docker] Update pip install
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16984/2/docker/bootstrap-python-env.sh
File docker/bootstrap-python-env.sh:

http://gerrit.cloudera.org:8080/#/c/16984/2/docker/bootstrap-python-env.sh@38
PS2, Line 38: standary
> nit: standard
Done



--
To view, visit http://gerrit.cloudera.org:8080/16984
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 16984
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 28 Jan 2021 15:37:26 +
Gerrit-HasComments: Yes


[kudu-CR] [docker] Update pip install

2021-01-28 Thread Grant Henke (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Bankim Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16984

to look at the new patch set (#3).

Change subject: [docker] Update pip install
..

[docker] Update pip install

It looks like Python has adjusted the get-pip.py URL to no longer
support Python 2.7. This patch updates the Docker scripts to
use the 2.7 specific URL and drop the old 2.6 speicific url now
that no supported images use Python 2.6.

Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
---
M docker/bootstrap-dev-env.sh
M docker/bootstrap-python-env.sh
2 files changed, 7 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/16984/3
--
To view, visit http://gerrit.cloudera.org:8080/16984
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 16984
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [docs] Add the 1.13.0 release notes to the prior release notes

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16996 )

Change subject: [docs] Add the 1.13.0 release notes to the prior release notes
..

[docs] Add the 1.13.0 release notes to the prior release notes

Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
Reviewed-on: http://gerrit.cloudera.org:8080/16996
Tested-by: Kudu Jenkins
Reviewed-by: Attila Bukor 
---
M docs/prior_release_notes.adoc
1 file changed, 197 insertions(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Attila Bukor: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16996
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
Gerrit-Change-Number: 16996
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) [docs] Fix 1.14 release notes title

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16995 )

Change subject: [docs] Fix 1.14 release notes title
..

[docs] Fix 1.14 release notes title

Change-Id: I61c768ccd400f1a0946793b7a8490ccf87b876d7
Reviewed-on: http://gerrit.cloudera.org:8080/16995
Tested-by: Kudu Jenkins
Reviewed-by: Attila Bukor 
---
M docs/release_notes.adoc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Attila Bukor: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16995
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I61c768ccd400f1a0946793b7a8490ccf87b876d7
Gerrit-Change-Number: 16995
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [docs] Update versions in docs to 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17000


Change subject: [docs] Update versions in docs to 1.14.0
..

[docs] Update versions in docs to 1.14.0

Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
---
M docs/administration.adoc
M docs/developing.adoc
M docs/quickstart.adoc
3 files changed, 9 insertions(+), 9 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/17000/1
--
To view, visit http://gerrit.cloudera.org:8080/17000
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If8ef66f9a989176bc84993fa264d3b78d184c5ea
Gerrit-Change-Number: 17000
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [examples] Update versions in examples to 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16999


Change subject: [examples] Update versions in examples to 1.14.0
..

[examples] Update versions in examples to 1.14.0

Change-Id: I0254bdd321c1313f1042e9ef5a9ad1fbbec42436
---
M examples/java/collectl/README.adoc
M examples/java/collectl/pom.xml
M examples/java/insert-loadgen/pom.xml
M examples/java/java-example/pom.xml
M examples/quickstart/nifi/README.adoc
M examples/quickstart/spark/README.adoc
M examples/quickstart/ycsb/README.adoc
M examples/scala/sbt-int-test-example/README.adoc
M examples/scala/sbt-int-test-example/build.sbt
M examples/scala/spark-example/pom.xml
10 files changed, 15 insertions(+), 15 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/16999/1
--
To view, visit http://gerrit.cloudera.org:8080/16999
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0254bdd321c1313f1042e9ef5a9ad1fbbec42436
Gerrit-Change-Number: 16999
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR](gh-pages) [blog] Add post announcing Kudu 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#2). ( 
http://gerrit.cloudera.org:8080/16998 )

Change subject: [blog] Add post announcing Kudu 1.14.0
..

[blog] Add post announcing Kudu 1.14.0

Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
---
A _posts/2021–01-28-apache-kudu-1-14-0-release.md
1 file changed, 80 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/98/16998/2
--
To view, visit http://gerrit.cloudera.org:8080/16998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Gerrit-Change-Number: 16998
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 


[kudu-CR](gh-pages) [blog] Add post announcing Kudu 1.14.0

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16998


Change subject: [blog] Add post announcing Kudu 1.14.0
..

[blog] Add post announcing Kudu 1.14.0

Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
---
A _posts/2021–01-28-apache-kudu-1-14-0-release.md
1 file changed, 80 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/98/16998/1
--
To view, visit http://gerrit.cloudera.org:8080/16998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idc1f6bc0c98151ffb250436d6c26701f8a0319e4
Gerrit-Change-Number: 16998
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [docs] Add the 1.14.0 release notes to the prior release notes

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16997


Change subject: [docs] Add the 1.14.0 release notes to the prior release notes
..

[docs] Add the 1.14.0 release notes to the prior release notes

Change-Id: If757e53b6c014f2c9f13988b746a758fa2b23cfa
---
M docs/prior_release_notes.adoc
1 file changed, 235 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/97/16997/1
--
To view, visit http://gerrit.cloudera.org:8080/16997
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If757e53b6c014f2c9f13988b746a758fa2b23cfa
Gerrit-Change-Number: 16997
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [docs] Add the 1.13.0 release notes to the prior release notes

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16996


Change subject: [docs] Add the 1.13.0 release notes to the prior release notes
..

[docs] Add the 1.13.0 release notes to the prior release notes

Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
---
M docs/prior_release_notes.adoc
1 file changed, 197 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/96/16996/1
--
To view, visit http://gerrit.cloudera.org:8080/16996
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7bd964cca7ec72b816eca994c914c9e932de6516
Gerrit-Change-Number: 16996
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR](branch-1.14.x) [docs] Fix 1.14 release notes title

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16995


Change subject: [docs] Fix 1.14 release notes title
..

[docs] Fix 1.14 release notes title

Change-Id: I61c768ccd400f1a0946793b7a8490ccf87b876d7
---
M docs/release_notes.adoc
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/16995/1
--
To view, visit http://gerrit.cloudera.org:8080/16995
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I61c768ccd400f1a0946793b7a8490ccf87b876d7
Gerrit-Change-Number: 16995
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR](branch-1.14.x) Bump version to 1.14.1-SNAPSHOT

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16994


Change subject: Bump version to 1.14.1-SNAPSHOT
..

Bump version to 1.14.1-SNAPSHOT

Change-Id: Id894844feac9fd8763e67f1bc1097256c6146bdd
---
M version.txt
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/94/16994/1
-- 
To view, visit http://gerrit.cloudera.org:8080/16994
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id894844feac9fd8763e67f1bc1097256c6146bdd
Gerrit-Change-Number: 16994
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [docker] Update pip install

2021-01-28 Thread Grant Henke (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Bankim Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16984

to look at the new patch set (#2).

Change subject: [docker] Update pip install
..

[docker] Update pip install

It looks like Python has adjusted the get-pip.py URL to no longer
support Python 2.7. This patch updates the Docker scripts to
use the 2.7 specific URL and drop the old 2.6 speicific url now
that no supported images use Python 2.6.

Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
---
M docker/bootstrap-dev-env.sh
M docker/bootstrap-python-env.sh
2 files changed, 7 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/16984/2
--
To view, visit http://gerrit.cloudera.org:8080/16984
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 16984
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [docker] Update pip install

2021-01-28 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16984 )

Change subject: [docker] Update pip install
..


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16984/1/docker/bootstrap-python-env.sh
File docker/bootstrap-python-env.sh:

http://gerrit.cloudera.org:8080/#/c/16984/1/docker/bootstrap-python-env.sh@37
PS1, Line 37: "7"
> Would it make sense to fail if below 2.7 in an obvious way? I imagine it'll
We could, though I haven't really made a habit of handling things we don't 
expect to happen at all in these scripts given all the base images we allow are 
guaranteed to have Python 2.7 or higher. For simplicity, I would rather not 
handle it.


http://gerrit.cloudera.org:8080/#/c/16984/1/docker/bootstrap-python-env.sh@38
PS1, Line 38:c
> nit: maybe mention that pypa's generic script isn't supported with 2.7 so w
Done


http://gerrit.cloudera.org:8080/#/c/16984/1/docker/bootstrap-python-env.sh@40
PS1, Line 40:  Python 2.
> nit: 2 and 3, as before?
Done



--
To view, visit http://gerrit.cloudera.org:8080/16984
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 16984
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 28 Jan 2021 14:32:21 +
Gerrit-HasComments: Yes


[kudu-CR](gh-pages) Update website for 1.14.0 release

2021-01-27 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16990


Change subject: Update website for 1.14.0 release
..

Update website for 1.14.0 release

Change-Id: Ia97cc8456a8ad0a81bb9732c7576641f41e165aa
---
M apidocs
M cpp-client-api
M docs
A releases/1.14.0/apidocs/allclasses-frame.html
A releases/1.14.0/apidocs/allclasses-noframe.html
A releases/1.14.0/apidocs/constant-values.html
A releases/1.14.0/apidocs/deprecated-list.html
A releases/1.14.0/apidocs/help-doc.html
A releases/1.14.0/apidocs/index-all.html
A releases/1.14.0/apidocs/index.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.ColumnSchemaBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.CompressionAlgorithm.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.Encoding.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnSchema.html
A 
releases/1.14.0/apidocs/org/apache/kudu/ColumnTypeAttributes.ColumnTypeAttributesBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/ColumnTypeAttributes.html
A releases/1.14.0/apidocs/org/apache/kudu/Schema.html
A releases/1.14.0/apidocs/org/apache/kudu/Type.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AbstractKuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AlterTableOptions.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AlterTableResponse.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduClient.AsyncKuduClientBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduClient.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.AsyncKuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.ReadMode.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.RowDataFormat.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduScanner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/AsyncKuduSession.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ColumnRangePredicate.html
A releases/1.14.0/apidocs/org/apache/kudu/client/CreateTableOptions.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Delete.html
A releases/1.14.0/apidocs/org/apache/kudu/client/DeleteIgnore.html
A releases/1.14.0/apidocs/org/apache/kudu/client/DeleteTableResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ExternalConsistencyMode.html
A releases/1.14.0/apidocs/org/apache/kudu/client/HasFailedRpcException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Insert.html
A releases/1.14.0/apidocs/org/apache/kudu/client/InsertIgnore.html
A releases/1.14.0/apidocs/org/apache/kudu/client/IsAlterTableDoneResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/IsCreateTableDoneResponse.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduClient.KuduClientBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduClient.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduException.OriginalException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduException.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduPartitioner.KuduPartitionerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPartitioner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPredicate.ComparisonOp.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduPredicate.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanToken.KuduScanTokenBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanToken.html
A 
releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanner.KuduScannerBuilder.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScanner.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduScannerIterator.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduSession.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTable.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTableStatistics.html
A releases/1.14.0/apidocs/org/apache/kudu/client/KuduTransaction.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ListTablesResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ListTabletServersResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/LocatedTablet.Replica.html
A releases/1.14.0/apidocs/org/apache/kudu/client/LocatedTablet.html
A releases/1.14.0/apidocs/org/apache/kudu/client/Operation.html
A releases/1.14.0/apidocs/org/apache/kudu/client/OperationResponse.html
A releases/1.14.0/apidocs/org/apache/kudu/client/PartialRow.html
A releases/1.14.0/apidocs/org/apache/kudu/client/PleaseThrottleException.html
A releases/1.14.0/apidocs/org/apache/kudu/client/RangePartitionBound.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ReplicaSelection.html
A releases/1.14.0/apidocs/org/apache/kudu/client/ResourceMetrics.html
A releases/1.14.0/apidocs/org/apache/kudu/client/RowError.html
A 

[kudu-CR](branch-1.14.x) KUDU-3239: [build] Disable errorprone on Java 11

2021-01-27 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16986


Change subject: KUDU-3239: [build] Disable errorprone on Java 11
..

KUDU-3239: [build] Disable errorprone on Java 11

It turns out that errorprone is not working with Java 11. This patch
disables it until we can upgrade/update the errorprone plugin to
work on with Java 11.

Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Reviewed-on: http://gerrit.cloudera.org:8080/16983
Reviewed-by: Bankim Bhavsar 
Reviewed-by: Hao Hao 
Reviewed-by: Alexey Serbin 
Tested-by: Kudu Jenkins
(cherry picked from commit 922ab9e7d46f683a5e04b4e36b1af9b95e96e025)
---
M java/gradle/quality.gradle
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/86/16986/1
--
To view, visit http://gerrit.cloudera.org:8080/16986
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Gerrit-Change-Number: 16986
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [docker] Update pip install

2021-01-27 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16984


Change subject: [docker] Update pip install
..

[docker] Update pip install

It looks like Python has adjusted the get-pip.py URL to no longer
support Python 2.7. This patch updates the Docker scripts to
use the 2.7 specific URL and drop the old 2.6 speicific url now
that no supported images use Python 2.6.

Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
---
M docker/bootstrap-dev-env.sh
M docker/bootstrap-python-env.sh
2 files changed, 5 insertions(+), 8 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/16984/1
--
To view, visit http://gerrit.cloudera.org:8080/16984
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia9a22e5084f8a5973d26372372f2ead4c46dcda9
Gerrit-Change-Number: 16984
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] KUDU-3239: [build] Disable errorprone on Java 11

2021-01-26 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16983 )

Change subject: KUDU-3239: [build] Disable errorprone on Java 11
..

KUDU-3239: [build] Disable errorprone on Java 11

It turns out that errorprone is not working with Java 11. This patch
disables it until we can upgrade/update the errorprone plugin to
work on with Java 11.

Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Reviewed-on: http://gerrit.cloudera.org:8080/16983
Reviewed-by: Bankim Bhavsar 
Reviewed-by: Hao Hao 
Reviewed-by: Alexey Serbin 
Tested-by: Kudu Jenkins
---
M java/gradle/quality.gradle
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Bankim Bhavsar: Looks good to me, approved
  Hao Hao: Looks good to me, approved
  Alexey Serbin: Looks good to me, approved
  Kudu Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/16983
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Gerrit-Change-Number: 16983
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-26 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16980 )

Change subject: Add release notes for 1.14.0
..

Add release notes for 1.14.0

Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Reviewed-on: http://gerrit.cloudera.org:8080/16980
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Greg Solovyev 
---
M docs/release_notes.adoc
1 file changed, 185 insertions(+), 29 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved
  Greg Solovyev: Looks good to me, but someone else must approve

--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 6
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 


[kudu-CR] KUDU-3239: [build] Disable errorprone on Java 11

2021-01-26 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Bankim Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16983

to look at the new patch set (#2).

Change subject: KUDU-3239: [build] Disable errorprone on Java 11
..

KUDU-3239: [build] Disable errorprone on Java 11

It turns out that errorprone is not working with Java 11. This patch
disables it until we can upgrade/update the errorprone plugin to
work on with Java 11.

Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
---
M java/gradle/quality.gradle
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/83/16983/2
--
To view, visit http://gerrit.cloudera.org:8080/16983
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Gerrit-Change-Number: 16983
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [build] Disable errorprone on Java 11

2021-01-26 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16983


Change subject: [build] Disable errorprone on Java 11
..

[build] Disable errorprone on Java 11

It turns out that errorprone is not working with Java 11. This patch
disables it until we can upgrade/update the errorprone plugin to
work on with Java 11.

Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
---
M java/gradle/quality.gradle
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/83/16983/1
--
To view, visit http://gerrit.cloudera.org:8080/16983
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic13f771222e67ac754f464f8015caf0a4901831d
Gerrit-Change-Number: 16983
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [build] Fix occasional Gradle build failures

2021-01-26 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16968 )

Change subject: [build] Fix occasional Gradle build failures
..

[build] Fix occasional Gradle build failures

Occasionally, on a clean build environment often as a part of the
CI pipline, the C++ build will fail with cryptic often Protobuf
related failures. It turns out this is a result of multiple
./gradlew calls which depend on the kudu-proto module.
This change fixes the issue by defining a single CMake target
to build the kudu-proto jar.

Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Reviewed-on: http://gerrit.cloudera.org:8080/16968
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Andrew Wong 
---
M CMakeLists.txt
M src/kudu/hms/CMakeLists.txt
M src/kudu/subprocess/CMakeLists.txt
3 files changed, 13 insertions(+), 2 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved
  Andrew Wong: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16968
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 16968
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Hello Mahesh Reddy, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, 
Greg Solovyev, Bankim Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16980

to look at the new patch set (#5).

Change subject: Add release notes for 1.14.0
..

Add release notes for 1.14.0

Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
---
M docs/release_notes.adoc
1 file changed, 185 insertions(+), 29 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/16980/5
--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 5
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16980 )

Change subject: Add release notes for 1.14.0
..


Patch Set 2:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@38
PS2, Line 38: they are at or near end-of-life
> This reads as Debian is also at EOL point, but I know there is Debian 10.7
Ah, I meant to put Debian 8 explicitly.


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@71
PS2, Line 71: jars
> nit: JARs ?
Done


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@74
PS2, Line 74: Id
> nit: identifier ?
It's called an Id in many places and that is commonly used short hand for 
identifier. I think I would like to keep it as is.


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@119
PS2, Line 119: CatalogManager
> +1: yes, I guess "Kudu Masters" is more user-friendly.
Done


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@175
PS2, Line 175: a table
> nit: the table ?
Actually I think it's redundant.


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@185
PS2, Line 185: jars
> nit: JARs ?
Done



--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 
Gerrit-Comment-Date: Tue, 26 Jan 2021 04:10:50 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Hello Mahesh Reddy, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, 
Greg Solovyev, Bankim Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16980

to look at the new patch set (#4).

Change subject: Add release notes for 1.14.0
..

Add release notes for 1.14.0

Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
---
M docs/release_notes.adoc
1 file changed, 185 insertions(+), 29 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/16980/4
--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16980 )

Change subject: Add release notes for 1.14.0
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@174
PS2, Line 174: * Fixed a heap-use-after-free bug in Kudu C++ client that might 
manifest itself when altering a
> Actually there isn't, though I am not sure if this bug was ever able to be
I see Alexey responded while I had a draft going. I will update with the Jira.



--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 
Gerrit-Comment-Date: Tue, 26 Jan 2021 03:24:03 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16980 )

Change subject: Add release notes for 1.14.0
..


Patch Set 2:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@55
PS2, Line 55: [[rn_1.14.0_new_features]]
> A few more worth considering:
Done


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@94
PS2, Line 94: The HMS
:   plugin will now check if the Kudu cluster associated with a 
table has HMS synchronization enabled.
> Should this be a separate bullet?
no, these are things that allow multiple clusters to be supported. I will 
adjust the language.


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@119
PS2, Line 119: CatalogManager
> nit: I thought we typically shied away from less user-facing names like thi
Done


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@139
PS2, Line 139: WrapSaslCall
> nit: would it make sense to not refer to specific function names and instea
Done


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@159
PS2, Line 159: maintenance manager
> nit: we should probably standardize this. "MaintenanceManager", as it is el
Done


http://gerrit.cloudera.org:8080/#/c/16980/2/docs/release_notes.adoc@174
PS2, Line 174: * Fixed a heap-use-after-free bug in Kudu C++ client that might 
manifest itself when altering a
> is there a Jira for this or any information on how this bug might manifest
Actually there isn't, though I am not sure if this bug was ever able to be seen 
by users in previous releases. I will ask Alexey to confirm. If not, we can 
remove the note.

https://github.com/apache/kudu/commit/ae45cd15c2703e3cfaf62aabb20ba79b961b3135



--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 
Gerrit-Comment-Date: Tue, 26 Jan 2021 03:21:02 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Hello Mahesh Reddy, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, 
Greg Solovyev, Bankim Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16980

to look at the new patch set (#3).

Change subject: Add release notes for 1.14.0
..

Add release notes for 1.14.0

Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
---
M docs/release_notes.adoc
1 file changed, 184 insertions(+), 29 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/16980/3
--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 


[kudu-CR] [build] Fix occasional Gradle build failures

2021-01-25 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16968

to look at the new patch set (#2).

Change subject: [build] Fix occasional Gradle build failures
..

[build] Fix occasional Gradle build failures

Occasionally, on a clean build environment often as a part of the
CI pipline, the C++ build will fail with cryptic often Protobuf
related failures. It turns out this is a result of multiple
./gradlew calls which depend on the kudu-proto module.
This change fixes the issue by defining a single CMake target
to build the kudu-proto jar.

Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
---
M CMakeLists.txt
M src/kudu/hms/CMakeLists.txt
M src/kudu/subprocess/CMakeLists.txt
3 files changed, 13 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/68/16968/2
--
To view, visit http://gerrit.cloudera.org:8080/16968
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 16968
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [build] Fix occasional Gradle build failures

2021-01-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16968 )

Change subject: [build] Fix occasional Gradle build failures
..


Patch Set 1:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/16968/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16968/1//COMMIT_MSG@10
PS1, Line 10: cryptic often Protobuf
> nit: maybe paste snippet of an exception?
I have seen a few different exceptions occur depending on the race/timing. 
Given this failure is the result of a recent change causing the potential for 
the race, I don't think it will be all that helpful of a discovery tool to note 
the various exceptions/failures we have seen so far.

https://gerrit.cloudera.org/#/c/16953/


http://gerrit.cloudera.org:8080/#/c/16968/1/CMakeLists.txt
File CMakeLists.txt:

http://gerrit.cloudera.org:8080/#/c/16968/1/CMakeLists.txt@123
PS1, Line 123: kudu-subprocess
> I'm a bit surprised that kudu-subprocess uses kudu-client mode.  Is that so
Yeah, I suppose kudu-proto is more accurate. I will fix it.


http://gerrit.cloudera.org:8080/#/c/16968/1/CMakeLists.txt@123
PS1, Line 123: kudu-hms
> nit: kudu-hive ?
Done


http://gerrit.cloudera.org:8080/#/c/16968/1/CMakeLists.txt@127
PS1, Line 127: add_custom_command(OUTPUT ${CLIENT_JAR}
 : COMMAND ./gradlew :kudu-client:jar ${GRADLE_FLAGS}
 : WORKING_DIRECTORY "${JAVA_DIR}"
 : DEPENDS init_gradle)
> Just making sure I understand this correctly. The race is that kudu-subproc
Yes your description is correct. For some reason I thought both modules used 
the client when initially building the patch. I adjusted it to use kudu-proto 
instead.



--
To view, visit http://gerrit.cloudera.org:8080/16968
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 16968
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 25 Jan 2021 20:51:17 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Hello Mahesh Reddy, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, 
Greg Solovyev, Bankim Bhavsar,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16980

to look at the new patch set (#2).

Change subject: Add release notes for 1.14.0
..

Add release notes for 1.14.0

Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
---
M docs/release_notes.adoc
1 file changed, 172 insertions(+), 29 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/16980/2
--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16980 )

Change subject: Add release notes for 1.14.0
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16980/1/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/16980/1/docs/release_notes.adoc@72
PS1, Line 72: ]
> extra bracket?
Done



--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy 
Gerrit-Comment-Date: Mon, 25 Jan 2021 20:30:10 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.14.x) Add release notes for 1.14.0

2021-01-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16980


Change subject: Add release notes for 1.14.0
..

Add release notes for 1.14.0

Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
---
M docs/release_notes.adoc
1 file changed, 172 insertions(+), 29 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/16980/1
--
To view, visit http://gerrit.cloudera.org:8080/16980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib8da030ce0f8797f8c366c56298fcda1020b00b6
Gerrit-Change-Number: 16980
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [java] address warnings from the SonarLint tool

2021-01-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16969 )

Change subject: [java] address warnings from the SonarLint tool
..


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/ColumnTypeAttributes.java
File java/kudu-client/src/main/java/org/apache/kudu/ColumnTypeAttributes.java:

http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/ColumnTypeAttributes.java@118
PS1, Line 118: return length == that.length;
I am not sure I prefer this syntax to the existing syntax. Do you think it's 
better? If so why?


http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java
File 
java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java:

http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java@51
PS1, Line 51: int numRows) {
I think reuseRowResult was kept around for compatibility concerns. It would be 
good to go back and validate.


http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/util/AsyncUtil.java
File java/kudu-client/src/main/java/org/apache/kudu/util/AsyncUtil.java:

http://gerrit.cloudera.org:8080/#/c/16969/1/java/kudu-client/src/main/java/org/apache/kudu/util/AsyncUtil.java@75
PS1, Line 75:   private AsyncUtil() {
Should this go at the top of the class? Here and elsewhere.



--
To view, visit http://gerrit.cloudera.org:8080/16969
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I306575b370fb69a54fdc24b9d770a99291828ef7
Gerrit-Change-Number: 16969
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Comment-Date: Mon, 25 Jan 2021 16:35:45 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Fix occasional Gradle build failures

2021-01-21 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16968


Change subject: [build] Fix occasional Gradle build failures
..

[build] Fix occasional Gradle build failures

Occasionally, on a clean build environment often as a part of the
CI pipline, the C++ build will fail with cryptic often Protobuf
related failures. It turns out this is a result of multiple
./gradlew calls which depend on the kudu-client and therefore
kudu-proto module. This change fixes the issue by defining a
single CMake target to build the kudu-client.

Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
---
M CMakeLists.txt
M src/kudu/hms/CMakeLists.txt
M src/kudu/subprocess/CMakeLists.txt
3 files changed, 13 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/68/16968/1
--
To view, visit http://gerrit.cloudera.org:8080/16968
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 16968
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-21 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16929 )

Change subject: KUDU-2612 Java client transaction implementation
..


Patch Set 4:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/16929/4/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
File java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java:

http://gerrit.cloudera.org:8080/#/c/16929/4/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java@504
PS4, Line 504: // We should have a UUID to construct ServerInfo for the 
master, but we have a chicken
Instead of duplicating the newMasterRpcProxy can we just call newMasterRpcProxy 
here?


http://gerrit.cloudera.org:8080/#/c/16929/4/java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java
File java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java:

http://gerrit.cloudera.org:8080/#/c/16929/4/java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java@292
PS4, Line 292: final Transactions.TxnStatePB txnState = resp.txnState();
Do we expose the transaction state publicly anywhere? It could be useful to get 
the state directly. Of course that means mapping the PB enum to a client side 
enum that is public (likely in the response object).


http://gerrit.cloudera.org:8080/#/c/16929/4/java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java@325
PS4, Line 325: synchronized (isInFlightSync) {
Would it make sense to update this first to prevent other call to things like 
`newKuduSession` immediately? Same for the other places.

Should we expect KuduTransaction to be threadsafe?


http://gerrit.cloudera.org:8080/#/c/16929/4/java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java@466
PS4, Line 466: // TODO(adar): By scheduling even the first RPC via timer, 
the sequence of
Is this comment copied somewhere? Is it completely relevant here?


http://gerrit.cloudera.org:8080/#/c/16929/4/java/kudu-client/src/main/java/org/apache/kudu/client/KuduTransaction.java@517
PS4, Line 517:   // TODO(aserbin): use proper names
Is this for a follow up?



--
To view, visit http://gerrit.cloudera.org:8080/16929
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie0236875e7264877c3f7a13da4a5a3da6423786b
Gerrit-Change-Number: 16929
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Thu, 21 Jan 2021 14:40:09 +
Gerrit-HasComments: Yes


[kudu-CR] KUDU-2612 Java client transaction implementation

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16929 )

Change subject: KUDU-2612 Java client transaction implementation
..


Patch Set 4:

I will look into this gradle ArrayIndexOutOfBoundsException failure. It seams 
to be only happening on pre-commit environments so far.


--
To view, visit http://gerrit.cloudera.org:8080/16929
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie0236875e7264877c3f7a13da4a5a3da6423786b
Gerrit-Change-Number: 16929
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Tue, 19 Jan 2021 18:19:31 +
Gerrit-HasComments: No


[kudu-CR](branch-1.14.x) Bump version to 1.14.0

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16962 )

Change subject: Bump version to 1.14.0
..

Bump version to 1.14.0

Change-Id: Id1c931f97db24d95d8780f9a4efd3688bf35fc54
Reviewed-on: http://gerrit.cloudera.org:8080/16962
Reviewed-by: Alexey Serbin 
Tested-by: Grant Henke 
---
M examples/java/collectl/README.adoc
M examples/java/collectl/pom.xml
M examples/java/insert-loadgen/pom.xml
M examples/java/java-example/pom.xml
M examples/quickstart/nifi/README.adoc
M examples/quickstart/spark/README.adoc
M examples/quickstart/ycsb/README.adoc
M examples/scala/sbt-int-test-example/README.adoc
M examples/scala/sbt-int-test-example/build.sbt
M examples/scala/spark-example/pom.xml
M version.txt
11 files changed, 16 insertions(+), 16 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Grant Henke: Verified

--
To view, visit http://gerrit.cloudera.org:8080/16962
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Id1c931f97db24d95d8780f9a4efd3688bf35fc54
Gerrit-Change-Number: 16962
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) Bump version to 1.14.0

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16962 )

Change subject: Bump version to 1.14.0
..


Patch Set 1: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/16962
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: comment
Gerrit-Change-Id: Id1c931f97db24d95d8780f9a4efd3688bf35fc54
Gerrit-Change-Number: 16962
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 19 Jan 2021 14:34:01 +
Gerrit-HasComments: No


[kudu-CR](branch-1.14.x) Bump version to 1.14.0

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change.

Change subject: Bump version to 1.14.0
..


Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/16962
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Id1c931f97db24d95d8780f9a4efd3688bf35fc54
Gerrit-Change-Number: 16962
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] Update version to 1.15.0-SNAPSHOT

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16961 )

Change subject: Update version to 1.15.0-SNAPSHOT
..


Patch Set 1: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/16961
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib81d6bed6524c733d8c4203d62ac51d570206613
Gerrit-Change-Number: 16961
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 19 Jan 2021 14:33:41 +
Gerrit-HasComments: No


[kudu-CR] Update version to 1.15.0-SNAPSHOT

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16961 )

Change subject: Update version to 1.15.0-SNAPSHOT
..

Update version to 1.15.0-SNAPSHOT

Change-Id: Ib81d6bed6524c733d8c4203d62ac51d570206613
Reviewed-on: http://gerrit.cloudera.org:8080/16961
Reviewed-by: Alexey Serbin 
Tested-by: Grant Henke 
---
M docs/release_notes.adoc
M version.txt
2 files changed, 29 insertions(+), 29 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Grant Henke: Verified

--
To view, visit http://gerrit.cloudera.org:8080/16961
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib81d6bed6524c733d8c4203d62ac51d570206613
Gerrit-Change-Number: 16961
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] Update version to 1.15.0-SNAPSHOT

2021-01-19 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change.

Change subject: Update version to 1.15.0-SNAPSHOT
..


Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/16961
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Ib81d6bed6524c733d8c4203d62ac51d570206613
Gerrit-Change-Number: 16961
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.14.x) Bump version to 1.14.0

2021-01-18 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16962


Change subject: Bump version to 1.14.0
..

Bump version to 1.14.0

Change-Id: Id1c931f97db24d95d8780f9a4efd3688bf35fc54
---
M examples/java/collectl/README.adoc
M examples/java/collectl/pom.xml
M examples/java/insert-loadgen/pom.xml
M examples/java/java-example/pom.xml
M examples/quickstart/nifi/README.adoc
M examples/quickstart/spark/README.adoc
M examples/quickstart/ycsb/README.adoc
M examples/scala/sbt-int-test-example/README.adoc
M examples/scala/sbt-int-test-example/build.sbt
M examples/scala/spark-example/pom.xml
M version.txt
11 files changed, 16 insertions(+), 16 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/16962/1
--
To view, visit http://gerrit.cloudera.org:8080/16962
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.14.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id1c931f97db24d95d8780f9a4efd3688bf35fc54
Gerrit-Change-Number: 16962
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] Update version to 1.15.0-SNAPSHOT

2021-01-18 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16961


Change subject: Update version to 1.15.0-SNAPSHOT
..

Update version to 1.15.0-SNAPSHOT

Change-Id: Ib81d6bed6524c733d8c4203d62ac51d570206613
---
M docs/release_notes.adoc
M version.txt
2 files changed, 29 insertions(+), 29 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/61/16961/1
--
To view, visit http://gerrit.cloudera.org:8080/16961
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib81d6bed6524c733d8c4203d62ac51d570206613
Gerrit-Change-Number: 16961
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR](gh-pages) [blog] Cross post Bloom filter predicate blog post

2021-01-17 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16957 )

Change subject: [blog] Cross post Bloom filter predicate blog post
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16957/1/_posts/2021-01-15-bloom-filter-predicate.md
File _posts/2021-01-15-bloom-filter-predicate.md:

http://gerrit.cloudera.org:8080/#/c/16957/1/_posts/2021-01-15-bloom-filter-predicate.md@20
PS1, Line 20: CDP Runtime 7.1.5
> If using these downstream product names and versions in an upstream blog, I
Given this is a cross post I think the edits to the post itself should be 
minimal or none if we can help it.

Perhaps just mapping the versions in the post to similar apache version in a 
note at the top would be good enough (as opposed to here inline).



--
To view, visit http://gerrit.cloudera.org:8080/16957
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I725c8cb4b10487db1d3d18046c3f9a48f0381923
Gerrit-Change-Number: 16957
Gerrit-PatchSet: 1
Gerrit-Owner: Bankim Bhavsar 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Reviewer: Wenzhe Zhou 
Gerrit-Comment-Date: Mon, 18 Jan 2021 04:45:06 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Add a test coverage report to the Java build

2021-01-17 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16954 )

Change subject: [build] Add a test coverage report to the Java build
..

[build] Add a test coverage report to the Java build

This patch adds a jacoco test coverage report to the Java build.
The report can be generated by running
`./gradlew jacocoAggregatedReport` and looking at
`java/build/reports/jacoco/jacocoAggregatedReport/html/index.html`

A follow on patch will add this coverage run to the build-and-test.sh
“COVERAGE” type build.

Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Reviewed-on: http://gerrit.cloudera.org:8080/16954
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M java/build.gradle
M java/buildSrc/build.gradle
M java/gradle/dependencies.gradle
M java/gradle/quality.gradle
4 files changed, 22 insertions(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16954
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Gerrit-Change-Number: 16954
Gerrit-PatchSet: 5
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [build] Separate protobuf java compilation into a seperate module

2021-01-17 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16953 )

Change subject: [build] Separate protobuf java compilation into a seperate 
module
..

[build] Separate protobuf java compilation into a seperate module

Currently we compile the protobufs in both the kudu-client
and the kudu-subprocess. Doing this also combines them onto
the same classpath as the module itself.

This patch breaks those classes into a separate `kudu-proto`
module so that we can avoid duplicate work and ignore
the `kudu-proto` module in API compatibility and test
coverage checks.

This module is not published, so there is no net change to the final
artifacts.

Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Reviewed-on: http://gerrit.cloudera.org:8080/16953
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M build-support/check_compatibility.py
M java/kudu-client/build.gradle
A java/kudu-proto/build.gradle
M java/kudu-subprocess/build.gradle
M java/settings.gradle
5 files changed, 46 insertions(+), 26 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](gh-pages) [blog] Cross post Bloom filter predicate blog post

2021-01-17 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16957 )

Change subject: [blog] Cross post Bloom filter predicate blog post
..


Patch Set 1: Verified+1 Code-Review+2


--
To view, visit http://gerrit.cloudera.org:8080/16957
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I725c8cb4b10487db1d3d18046c3f9a48f0381923
Gerrit-Change-Number: 16957
Gerrit-PatchSet: 1
Gerrit-Owner: Bankim Bhavsar 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Reviewer: Wenzhe Zhou 
Gerrit-Comment-Date: Mon, 18 Jan 2021 00:51:57 +
Gerrit-HasComments: No


[kudu-CR] [build] Separate protobuf java compilation into a seperate module

2021-01-17 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16953 )

Change subject: [build] Separate protobuf java compilation into a seperate 
module
..


Patch Set 3:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/16953/2/build-support/check_compatibility.py
File build-support/check_compatibility.py:

http://gerrit.cloudera.org:8080/#/c/16953/2/build-support/check_compatibility.py@135
PS2, Line 135: kudu-pro
> kudu-proto ?
Done


http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-client/build.gradle
File java/kudu-client/build.gradle:

http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-client/build.gradle@22
PS2, Line 22: compile
> Just to make sure: this will result in shading this dependency in kudu-clie
yes, compileUnshaded is required to explicitly "unshade" a dependency.


http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-client/build.gradle@35
PS2, Line 35:
> Do we need to specify this explicitly once moving kudu-proto into a separat
this isn't explicitly required because it is pulled in via kudu-proto. I will 
remove it.


http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-proto/build.gradle
File java/kudu-proto/build.gradle:

http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-proto/build.gradle@35
PS2, Line 35: kudu-pro
> kudu-proto
Done


http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-proto/build.gradle@40
PS2, Line 40: kudu-pro
> kudu-proto
Done


http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-subprocess/build.gradle
File java/kudu-subprocess/build.gradle:

http://gerrit.cloudera.org:8080/#/c/16953/2/java/kudu-subprocess/build.gradle@32
PS2, Line 32:   compile libs.protobufJavaUtil
:
> I'm not an expert in gradle, so just wanted to clarify: wouldn't these be i
protobufJava will, I will remove it.



--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 Jan 2021 00:19:29 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Separate protobuf java compilation into a seperate module

2021-01-17 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16953

to look at the new patch set (#3).

Change subject: [build] Separate protobuf java compilation into a seperate 
module
..

[build] Separate protobuf java compilation into a seperate module

Currently we compile the protobufs in both the kudu-client
and the kudu-subprocess. Doing this also combines them onto
the same classpath as the module itself.

This patch breaks those classes into a separate `kudu-proto`
module so that we can avoid duplicate work and ignore
the `kudu-proto` module in API compatibility and test
coverage checks.

This module is not published, so there is no net change to the final
artifacts.

Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
---
M build-support/check_compatibility.py
M java/kudu-client/build.gradle
A java/kudu-proto/build.gradle
M java/kudu-subprocess/build.gradle
M java/settings.gradle
5 files changed, 46 insertions(+), 26 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/16953/3
--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [build] Add a test coverage report to the Java build

2021-01-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16954 )

Change subject: [build] Add a test coverage report to the Java build
..


Patch Set 2:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16954/2/java/gradle/quality.gradle
File java/gradle/quality.gradle:

http://gerrit.cloudera.org:8080/#/c/16954/2/java/gradle/quality.gradle@104
PS2, Line 104: kudu-rpc
> I guess it should be kudu-proto as per PS2 of https://gerrit.cloudera.org/#
good catch


http://gerrit.cloudera.org:8080/#/c/16954/2/java/gradle/quality.gradle@106
PS2, Line 106: kudu-rpc
> ditto
Done


http://gerrit.cloudera.org:8080/#/c/16954/2/java/gradle/quality.gradle@114
PS2, Line 114: true
> Just curious: for what purpose we need the report in XML format?
This is useful when picked up by various reporting tools in Jenkins jobs.



--
To view, visit http://gerrit.cloudera.org:8080/16954
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Gerrit-Change-Number: 16954
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 15 Jan 2021 22:21:34 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Add a test coverage report to the Java build

2021-01-15 Thread Grant Henke (Code Review)
Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16954

to look at the new patch set (#3).

Change subject: [build] Add a test coverage report to the Java build
..

[build] Add a test coverage report to the Java build

This patch adds a jacoco test coverage report to the Java build.
The report can be generated by running
`./gradlew jacocoAggregatedReport` and looking at
`java/build/reports/jacoco/jacocoAggregatedReport/html/index.html`

A follow on patch will add this coverage run to the build-and-test.sh
“COVERAGE” type build.

Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
---
M java/build.gradle
M java/buildSrc/build.gradle
M java/gradle/dependencies.gradle
M java/gradle/quality.gradle
4 files changed, 22 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/16954/3
--
To view, visit http://gerrit.cloudera.org:8080/16954
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Gerrit-Change-Number: 16954
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [build] Add a test coverage report to the Java build

2021-01-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16954 )

Change subject: [build] Add a test coverage report to the Java build
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16954/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16954/1//COMMIT_MSG@11
PS1, Line 11: gradlew
> Thanks for the suggestion! That worked, and interestingly enough it still o
yeah, it likely reported on that tests that did run in your initial run.



--
To view, visit http://gerrit.cloudera.org:8080/16954
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Gerrit-Change-Number: 16954
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 15 Jan 2021 19:26:47 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Add a test coverage report to the Java build

2021-01-15 Thread Grant Henke (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16954

to look at the new patch set (#2).

Change subject: [build] Add a test coverage report to the Java build
..

[build] Add a test coverage report to the Java build

This patch adds a jacoco test coverage report to the Java build.
The report can be generated by running
`./gradlew jacocoAggregatedReport` and looking at
`java/build/reports/jacoco/jacocoAggregatedReport/html/index.html`

A follow on patch will add this coverage run to the build-and-test.sh
“COVERAGE” type build.

Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
---
M java/build.gradle
M java/buildSrc/build.gradle
M java/gradle/dependencies.gradle
M java/gradle/quality.gradle
4 files changed, 22 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/16954/2
--
To view, visit http://gerrit.cloudera.org:8080/16954
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Gerrit-Change-Number: 16954
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [build] Add a test coverage report to the Java build

2021-01-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16954 )

Change subject: [build] Add a test coverage report to the Java build
..


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/16954/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16954/1//COMMIT_MSG@9
PS1, Line 9: t.
> nit: build
Done


http://gerrit.cloudera.org:8080/#/c/16954/1//COMMIT_MSG@11
PS1, Line 11: gradlew
> Any pointers for why running this might not result in a java/build director
Likely because some tests failed. It won't generate the report if tests fail. 
If you want to test that theory you can run `./gradlew jacocoAggregatedReport 
-x test` to skip running the tests which will result in a generated report, but 
with 0% coverage given no tests actually ran.



--
To view, visit http://gerrit.cloudera.org:8080/16954
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Gerrit-Change-Number: 16954
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 15 Jan 2021 15:02:10 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Separate protobuf java compilation into a seperate module

2021-01-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16953 )

Change subject: [build] Separate protobuf java compilation into a seperate 
module
..


Patch Set 1:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/16953/1//COMMIT_MSG
Commit Message:

PS1:
> I'm curious how you spotted this. Just noticed it going through build logs?
Done


PS1:
> Nevermind, I see it from the code coverage test.
Done


http://gerrit.cloudera.org:8080/#/c/16953/1//COMMIT_MSG@7
PS1, Line 7: [build] Seperate protobuf java compilation into a seperate module
> nit: separate, same elsewhere
Done


http://gerrit.cloudera.org:8080/#/c/16953/1//COMMIT_MSG@15
PS1, Line 15: the `kudu-rpc` module in API compatibility and test
> It's true that it's the most accurate for now, but isn't it possible that w
The jar is not published, so we can change the name at any time. I will go with 
kudu-proto for now.



--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 15 Jan 2021 15:00:27 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Separate protobuf java compilation into a seperate module

2021-01-15 Thread Grant Henke (Code Review)
Hello Attila Bukor, Kudu Jenkins, Andrew Wong,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/16953

to look at the new patch set (#2).

Change subject: [build] Separate protobuf java compilation into a seperate 
module
..

[build] Separate protobuf java compilation into a seperate module

Currently we compile the protobufs in both the kudu-client
and the kudu-subprocess. Doing this also combines them onto
the same classpath as the module itself.

This patch breaks those classes into a separate `kudu-proto`
module so that we can avoid duplicate work and ignore
the `kudu-proto` module in API compatibility and test
coverage checks.

This module is not published, so there is no net change to the final
artifacts.

Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
---
M build-support/check_compatibility.py
M java/kudu-client/build.gradle
A java/kudu-proto/build.gradle
M java/kudu-subprocess/build.gradle
M java/settings.gradle
5 files changed, 46 insertions(+), 24 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/16953/2
--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] KUDU-2612 Java client transaction API

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16894 )

Change subject: KUDU-2612 Java client transaction API
..


Patch Set 7: Code-Review+2

Looks good to me. Thanks for revising.


--
To view, visit http://gerrit.cloudera.org:8080/16894
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idbb18e1ac0454a8ef9e3486430dfaa336e381e07
Gerrit-Change-Number: 16894
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Fri, 15 Jan 2021 01:31:41 +
Gerrit-HasComments: No


[kudu-CR] [build] Seperate protobuf java compilation into a seperate module

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16953 )

Change subject: [build] Seperate protobuf java compilation into a seperate 
module
..


Patch Set 1: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 14 Jan 2021 21:52:38 +
Gerrit-HasComments: No


[kudu-CR] [build] Seperate protobuf java compilation into a seperate module

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change.

Change subject: [build] Seperate protobuf java compilation into a seperate 
module
..


Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [client] updated multi-row transaction API

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16948 )

Change subject: [client] updated multi-row transaction API
..


Patch Set 3: Code-Review+2


--
To view, visit http://gerrit.cloudera.org:8080/16948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id646c41d60f385f079ebfaec2daf7c2a108306b2
Gerrit-Change-Number: 16948
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 14 Jan 2021 21:03:50 +
Gerrit-HasComments: No


[kudu-CR] [build] Seperate protobuf java compilation into a seperate module

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16953 )

Change subject: [build] Seperate protobuf java compilation into a seperate 
module
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16953/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16953/1//COMMIT_MSG@15
PS1, Line 15: the `kudu-rpc` module in API compatibility and test
A better name for this might be `kudu-proto` or `kudu-gen` given more code is 
generated than just rpc code. I am open to suggestions.



--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 14 Jan 2021 20:36:59 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Seperate protobuf java compilation into a seperate module

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16953


Change subject: [build] Seperate protobuf java compilation into a seperate 
module
..

[build] Seperate protobuf java compilation into a seperate module

Currently we compile the RPC Protobufs in both the kudu-client
and the kudu-subprocess. Doing this also combines them onto
the same classpath as the module itself.

This patch breaks those classes into a seperate `kudu-rpc`
module so that we can avoid duplicate work and ignore
the `kudu-rpc` module in API compatibility and test
coverage checks.

This module is not published, so there is no net change to the final
artifacts.

Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
---
M build-support/check_compatibility.py
M java/kudu-client/build.gradle
A java/kudu-rpc/build.gradle
M java/kudu-subprocess/build.gradle
M java/settings.gradle
5 files changed, 46 insertions(+), 24 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/16953/1
--
To view, visit http://gerrit.cloudera.org:8080/16953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d
Gerrit-Change-Number: 16953
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [build] Add a test coverage report to the Java build

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16954


Change subject: [build] Add a test coverage report to the Java build
..

[build] Add a test coverage report to the Java build

This patch adds a jacoco test coverage report to the Java built.
The report can be generated by running
`./gradlew jacocoAggregatedReport` and looking at
`java/build/reports/jacoco/jacocoAggregatedReport/html/index.html`

A follow on patch will add this coverage run to the build-and-test.sh
“COVERAGE” type build.

Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
---
M java/build.gradle
M java/buildSrc/build.gradle
M java/gradle/dependencies.gradle
M java/gradle/quality.gradle
4 files changed, 22 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/16954/1
--
To view, visit http://gerrit.cloudera.org:8080/16954
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0f2a2b1836b5ade72a757a8d7e2c518858ec48bb
Gerrit-Change-Number: 16954
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 


[kudu-CR] [client] updated multi-row transaction API

2021-01-14 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16948 )

Change subject: [client] updated multi-row transaction API
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16948/1/src/kudu/client/client.h
File src/kudu/client/client.h:

http://gerrit.cloudera.org:8080/#/c/16948/1/src/kudu/client/client.h@557
PS1, Line 557:  Serialize(s
> I wanted to be able to call this method as just Serialize(_token), so t
Can we have Serialize(options, _token) and Serialize(_token)? I prefer 
the options as an argument as well.



--
To view, visit http://gerrit.cloudera.org:8080/16948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id646c41d60f385f079ebfaec2daf7c2a108306b2
Gerrit-Change-Number: 16948
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 14 Jan 2021 14:39:43 +
Gerrit-HasComments: Yes


[kudu-CR] [build] Add more options to the compatibility checker script

2021-01-13 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16943 )

Change subject: [build] Add more options to the compatibility checker script
..

[build] Add more options to the compatibility checker script

This patch adjusts the argument parsing and adds more argument
options to the compatibility checker script. These options are useful
for filtering down to various report granularities and issues.

Change-Id: I1fa93bd069b4412266d3cafb00e1d49f578f0516
Reviewed-on: http://gerrit.cloudera.org:8080/16943
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M build-support/check_compatibility.py
1 file changed, 45 insertions(+), 30 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16943
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I1fa93bd069b4412266d3cafb00e1d49f578f0516
Gerrit-Change-Number: 16943
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Kudu Jenkins (120)


<    1   2   3   4   5   6   7   8   9   10   >