[kudu-CR] [docs] add blog post announcement as part of a release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15951 )

Change subject: [docs] add blog post announcement as part of a release
..

[docs] add blog post announcement as part of a release

Change-Id: I9bcb826f671ee7878b6e7a66ed9a77256847a153
Reviewed-on: http://gerrit.cloudera.org:8080/15951
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M RELEASING.adoc
1 file changed, 3 insertions(+), 0 deletions(-)

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

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I9bcb826f671ee7878b6e7a66ed9a77256847a153
Gerrit-Change-Number: 15951
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [examples] use org.apache.kudu packages v1.12.0

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15953 )

Change subject: [examples] use org.apache.kudu packages v1.12.0
..

[examples] use org.apache.kudu packages v1.12.0

Change-Id: I988e48b3ffdd1fca016268a3d01dab2815e6eb3d
Reviewed-on: http://gerrit.cloudera.org:8080/15743
Reviewed-by: Alexey Serbin 
Tested-by: Kudu Jenkins
(cherry picked from commit e2acd00a3727b0c434e095855aaca49d2797b98d)
Reviewed-on: http://gerrit.cloudera.org:8080/15953
Reviewed-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/scala/sbt-int-test-example/README.adoc
M examples/scala/sbt-int-test-example/build.sbt
M examples/scala/spark-example/pom.xml
9 files changed, 14 insertions(+), 14 deletions(-)

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

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I988e48b3ffdd1fca016268a3d01dab2815e6eb3d
Gerrit-Change-Number: 15953
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [docs] add blog post announcement as part of a release

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15951 )

Change subject: [docs] add blog post announcement as part of a release
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9bcb826f671ee7878b6e7a66ed9a77256847a153
Gerrit-Change-Number: 15951
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 19 May 2020 03:25:23 +
Gerrit-HasComments: No


[kudu-CR] [examples] use org.apache.kudu packages v1.12.0

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15953 )

Change subject: [examples] use org.apache.kudu packages v1.12.0
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I988e48b3ffdd1fca016268a3d01dab2815e6eb3d
Gerrit-Change-Number: 15953
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 19 May 2020 03:21:37 +
Gerrit-HasComments: No


[kudu-CR] KUDU-2817: Upgrade snappy to 1.1.8

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15927 )

Change subject: KUDU-2817: Upgrade snappy to 1.1.8
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15927/2/thirdparty/build-definitions.sh
File thirdparty/build-definitions.sh:

http://gerrit.cloudera.org:8080/#/c/15927/2/thirdparty/build-definitions.sh@528
PS2, Line 528: -DSNAPPY_BUILD_TESTS=OFF
> I saw a TSAN instrumentation issue in the test code, but ideally we should
I see, thank you for the information.  Yes, I agree that building tests in 
third-party components is not needed.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I19e4eda122c0ea957bbe504fc7de681bb72068cd
Gerrit-Change-Number: 15927
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 19 May 2020 03:11:32 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.12.x) [version.txt] update version to 1.12.1-SNAPSHOT

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15952 )

Change subject: [version.txt] update version to 1.12.1-SNAPSHOT
..

[version.txt] update version to 1.12.1-SNAPSHOT

Change-Id: I1f78eb1aeb558684722d1c953a8ad9d16cbc3022
Reviewed-on: http://gerrit.cloudera.org:8080/15952
Reviewed-by: Grant Henke 
Tested-by: Kudu Jenkins
---
M version.txt
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Grant Henke: Looks good to me, approved
  Kudu Jenkins: Verified

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I1f78eb1aeb558684722d1c953a8ad9d16cbc3022
Gerrit-Change-Number: 15952
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [docs] fix incorrect branch name in README.adoc

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15949 )

Change subject: [docs] fix incorrect branch name in README.adoc
..

[docs] fix incorrect branch name in README.adoc

Change-Id: Ib2eafae07f00f44f8891531b4515fc0a59641413
Reviewed-on: http://gerrit.cloudera.org:8080/15949
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke 
---
M README.adoc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Grant Henke: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2eafae07f00f44f8891531b4515fc0a59641413
Gerrit-Change-Number: 15949
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [docs] upgrade liquid version

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15950 )

Change subject: [docs] upgrade liquid version
..

[docs] upgrade liquid version

Current docs support scripts use liquid 3.0.6. which can cause the
following error:
  jekyll 3.6.3 | Error:  undefined method `to_date' for Liquid::Utils:Module

Upgrade the versio to be 4.0.0 as suggested in [1].

[1]: https://github.com/poole/hyde/issues/215

Change-Id: Ie5220a888b4bac482f11e908d892f5d3cfc58d3f
Reviewed-on: http://gerrit.cloudera.org:8080/15950
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke 
---
M docs/support/scripts/Gemfile.lock
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Grant Henke: Looks good to me, approved

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

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


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15946 )

Change subject: [site] add 1.12.0 release post
..

[site] add 1.12.0 release post

Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Reviewed-on: http://gerrit.cloudera.org:8080/15946
Reviewed-by: Grant Henke 
Tested-by: Grant Henke 
---
A _posts/2020-05-18-apache-kudu-1-12-0-release.md
1 file changed, 67 insertions(+), 0 deletions(-)

Approvals:
  Grant Henke: Looks good to me, approved; Verified

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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: merged
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 3
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 


[kudu-CR] [examples] use org.apache.kudu packages v1.12.0

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15953 )

Change subject: [examples] use org.apache.kudu packages v1.12.0
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I988e48b3ffdd1fca016268a3d01dab2815e6eb3d
Gerrit-Change-Number: 15953
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 23:53:25 +
Gerrit-HasComments: No


[kudu-CR] [docs] fix incorrect branch name in README.adoc

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15949 )

Change subject: [docs] fix incorrect branch name in README.adoc
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib2eafae07f00f44f8891531b4515fc0a59641413
Gerrit-Change-Number: 15949
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 23:47:19 +
Gerrit-HasComments: No


[kudu-CR] [docs] upgrade liquid version

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15950 )

Change subject: [docs] upgrade liquid version
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie5220a888b4bac482f11e908d892f5d3cfc58d3f
Gerrit-Change-Number: 15950
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 23:47:02 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15946 )

Change subject: [site] add 1.12.0 release post
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 23:46:14 +
Gerrit-HasComments: No


[kudu-CR](branch-1.12.x) [version.txt] update version to 1.12.1-SNAPSHOT

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15952 )

Change subject: [version.txt] update version to 1.12.1-SNAPSHOT
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I1f78eb1aeb558684722d1c953a8ad9d16cbc3022
Gerrit-Change-Number: 15952
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 23:46:31 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15946 )

Change subject: [site] add 1.12.0 release post
..


Patch Set 2: Verified+1


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 23:46:20 +
Gerrit-HasComments: No


[kudu-CR] [examples] use org.apache.kudu packages v1.12.0

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15953


Change subject: [examples] use org.apache.kudu packages v1.12.0
..

[examples] use org.apache.kudu packages v1.12.0

Change-Id: I988e48b3ffdd1fca016268a3d01dab2815e6eb3d
Reviewed-on: http://gerrit.cloudera.org:8080/15743
Reviewed-by: Alexey Serbin 
Tested-by: Kudu Jenkins
(cherry picked from commit e2acd00a3727b0c434e095855aaca49d2797b98d)
---
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/scala/sbt-int-test-example/README.adoc
M examples/scala/sbt-int-test-example/build.sbt
M examples/scala/spark-example/pom.xml
9 files changed, 14 insertions(+), 14 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I988e48b3ffdd1fca016268a3d01dab2815e6eb3d
Gerrit-Change-Number: 15953
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15946 )

Change subject: [site] add 1.12.0 release post
..


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/15946/1/_posts/2020-05-11-apache-kudu-1-12-0-release.md
File _posts/2020-05-11-apache-kudu-1-12-0-release.md:

PS1:
> I think we should date this post for today given that is when we are "annou
Done


http://gerrit.cloudera.org:8080/#/c/15946/1/_posts/2020-05-11-apache-kudu-1-12-0-release.md@66
PS1, Line 66:
> nit: add a comma after 'Additionally' ?
Done



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 23:15:31 +
Gerrit-HasComments: Yes


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Grant Henke,

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

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

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

Change subject: [site] add 1.12.0 release post
..

[site] add 1.12.0 release post

Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
---
A _posts/2020-05-18-apache-kudu-1-12-0-release.md
1 file changed, 67 insertions(+), 0 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 


[kudu-CR](branch-1.12.x) [version.txt] update version to 1.12.1-SNAPSHOT

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15952


Change subject: [version.txt] update version to 1.12.1-SNAPSHOT
..

[version.txt] update version to 1.12.1-SNAPSHOT

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



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1f78eb1aeb558684722d1c953a8ad9d16cbc3022
Gerrit-Change-Number: 15952
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15946 )

Change subject: [site] add 1.12.0 release post
..


Patch Set 1: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15946/1/_posts/2020-05-11-apache-kudu-1-12-0-release.md
File _posts/2020-05-11-apache-kudu-1-12-0-release.md:

http://gerrit.cloudera.org:8080/#/c/15946/1/_posts/2020-05-11-apache-kudu-1-12-0-release.md@66
PS1, Line 66: Additionally experimental Docker images are published to
nit: add a comma after 'Additionally' ?



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 22:58:39 +
Gerrit-HasComments: Yes


[kudu-CR](gh-pages) [site] update docs, apidocs and cpp-client-api links for Kudu 1.12.0 release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15948 )

Change subject: [site] update docs, apidocs and cpp-client-api links for Kudu 
1.12.0 release
..


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


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I0ac62ce4bd29bb3b37fd6b1b8dcbab08278cc788
Gerrit-Change-Number: 15948
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 22:56:09 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) [site] update docs, apidocs and cpp-client-api links for Kudu 1.12.0 release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15948 )

Change subject: [site] update docs, apidocs and cpp-client-api links for Kudu 
1.12.0 release
..

[site] update docs, apidocs and cpp-client-api links for Kudu 1.12.0 release

Change-Id: I0ac62ce4bd29bb3b37fd6b1b8dcbab08278cc788
Reviewed-on: http://gerrit.cloudera.org:8080/15948
Reviewed-by: Hao Hao 
Tested-by: Hao Hao 
---
M apidocs
M cpp-client-api
M docs
3 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Hao Hao: Looks good to me, approved; Verified

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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: merged
Gerrit-Change-Id: I0ac62ce4bd29bb3b37fd6b1b8dcbab08278cc788
Gerrit-Change-Number: 15948
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 


[kudu-CR] [docs] add blog post announcement as part of a release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15951


Change subject: [docs] add blog post announcement as part of a release
..

[docs] add blog post announcement as part of a release

Change-Id: I9bcb826f671ee7878b6e7a66ed9a77256847a153
---
M RELEASING.adoc
1 file changed, 3 insertions(+), 0 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9bcb826f671ee7878b6e7a66ed9a77256847a153
Gerrit-Change-Number: 15951
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 


[kudu-CR] [docs] upgrade liquid version

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15950


Change subject: [docs] upgrade liquid version
..

[docs] upgrade liquid version

Current docs support scripts use liquid 3.0.6. which can cause the
following error:
  jekyll 3.6.3 | Error:  undefined method `to_date' for Liquid::Utils:Module

Upgrade the versio to be 4.0.0 as suggested in [1].

[1]: https://github.com/poole/hyde/issues/215

Change-Id: Ie5220a888b4bac482f11e908d892f5d3cfc58d3f
---
M docs/support/scripts/Gemfile.lock
1 file changed, 1 insertion(+), 1 deletion(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie5220a888b4bac482f11e908d892f5d3cfc58d3f
Gerrit-Change-Number: 15950
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 


[kudu-CR] [docs] fix incorrect branch name in README.adoc

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15949


Change subject: [docs] fix incorrect branch name in README.adoc
..

[docs] fix incorrect branch name in README.adoc

Change-Id: Ib2eafae07f00f44f8891531b4515fc0a59641413
---
M README.adoc
1 file changed, 1 insertion(+), 1 deletion(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib2eafae07f00f44f8891531b4515fc0a59641413
Gerrit-Change-Number: 15949
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 


[kudu-CR] [client] Remove allocator param for direct Bloom filter predicate

2020-05-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15947 )

Change subject: [client] Remove allocator param for direct Bloom filter 
predicate
..


Patch Set 2:

(1 comment)

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

http://gerrit.cloudera.org:8080/#/c/15947/1/src/kudu/client/client.h@1154
PS1, Line 1154:   KuduPredicate* NewInBloomFilterPredicate(const Slice& 
col_name,
  :const 
std::vector& bloom_filters);
IDE sticking & to variable name on Mac. Configured it for future.
See PS2.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7b57a6cacc8a5b6243787df8f896b74aa04f3916
Gerrit-Change-Number: 15947
Gerrit-PatchSet: 2
Gerrit-Owner: Bankim Bhavsar 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 22:38:48 +
Gerrit-HasComments: Yes


[kudu-CR] [client] Remove allocator param for direct Bloom filter predicate

2020-05-18 Thread Bankim Bhavsar (Code Review)
Hello Kudu Jenkins,

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

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

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

Change subject: [client] Remove allocator param for direct Bloom filter 
predicate
..

[client] Remove allocator param for direct Bloom filter predicate

Direct Bloom filter predicate in kudu client API is meant for consumption by
Impala.

Supplying the allocator that's associated with Bloom filter is turning out to
be tricky for Impala. For instance, it doesn't want to support Clone() and
concerned with the allocator being used by kudu client for
allocation/deallocation of memory.

Allocator associated with the Bloom filter is only used for cloning a predicate
and the support for cloning is only used for internal testing purposes.

So this change removes the allocator parameter from the direct Bloom filter
predicate API. For cloning the direct Bloom filter, it uses the default
allocator instead.

Since this API is marked as advanced/unstable, no compatibility concerns.

Change-Id: I7b57a6cacc8a5b6243787df8f896b74aa04f3916
---
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/predicate-test.cc
M src/kudu/client/scan_predicate-internal.h
M src/kudu/client/scan_predicate.cc
5 files changed, 18 insertions(+), 40 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7b57a6cacc8a5b6243787df8f896b74aa04f3916
Gerrit-Change-Number: 15947
Gerrit-PatchSet: 2
Gerrit-Owner: Bankim Bhavsar 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](gh-pages) [site] update docs, apidocs and cpp-client-api links for Kudu 1.12.0 release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15948


Change subject: [site] update docs, apidocs and cpp-client-api links for Kudu 
1.12.0 release
..

[site] update docs, apidocs and cpp-client-api links for Kudu 1.12.0 release

Change-Id: I0ac62ce4bd29bb3b37fd6b1b8dcbab08278cc788
---
M apidocs
M cpp-client-api
M docs
3 files changed, 3 insertions(+), 3 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0ac62ce4bd29bb3b37fd6b1b8dcbab08278cc788
Gerrit-Change-Number: 15948
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 


[kudu-CR] [client] Remove allocator param for direct Bloom filter predicate

2020-05-18 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15947


Change subject: [client] Remove allocator param for direct Bloom filter 
predicate
..

[client] Remove allocator param for direct Bloom filter predicate

Direct Bloom filter predicate in kudu client API is meant for consumption by
Impala.

Supplying the allocator that's associated with Bloom filter is turning out to
be tricky for Impala. For instance, it doesn't want to support Clone() and
concerned with the allocator being used by kudu client for
allocation/deallocation of memory.

Allocator associated with the Bloom filter is only used for cloning a predicate
and the support for cloning is only used for internal testing purposes.

So this change removes the allocator parameter from the direct Bloom filter
predicate API. For cloning the direct Bloom filter, it uses the default
allocator instead.

Since this API is marked as advanced/unstable, no compatibility concerns.

Change-Id: I7b57a6cacc8a5b6243787df8f896b74aa04f3916
---
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/predicate-test.cc
M src/kudu/client/scan_predicate-internal.h
M src/kudu/client/scan_predicate.cc
5 files changed, 22 insertions(+), 44 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7b57a6cacc8a5b6243787df8f896b74aa04f3916
Gerrit-Change-Number: 15947
Gerrit-PatchSet: 1
Gerrit-Owner: Bankim Bhavsar 


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15946 )

Change subject: [site] add 1.12.0 release post
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15946/1/_posts/2020-05-11-apache-kudu-1-12-0-release.md
File _posts/2020-05-11-apache-kudu-1-12-0-release.md:

PS1:
I think we should date this post for today given that is when we are 
"announcing" it.



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 22:20:34 +
Gerrit-HasComments: Yes


[kudu-CR](gh-pages) [site] add 1.12.0 release post

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15946


Change subject: [site] add 1.12.0 release post
..

[site] add 1.12.0 release post

Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
---
A _posts/2020-05-11-apache-kudu-1-12-0-release.md
1 file changed, 67 insertions(+), 0 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newchange
Gerrit-Change-Id: If84f8412fae3796ddb49fa48317bc9e0479a485e
Gerrit-Change-Number: 15946
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 


[kudu-CR](gh-pages) [site] add more docs for Kudu 1.12.0 release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15945 )

Change subject: [site] add more docs for Kudu 1.12.0 release
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: If7fa629219c77907d49371d1754103d6286677a2
Gerrit-Change-Number: 15945
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 21:25:54 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) [site] add more docs for Kudu 1.12.0 release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15945 )

Change subject: [site] add more docs for Kudu 1.12.0 release
..


Patch Set 1: Verified+1


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: If7fa629219c77907d49371d1754103d6286677a2
Gerrit-Change-Number: 15945
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Hao Hao 
Gerrit-Comment-Date: Mon, 18 May 2020 21:25:56 +
Gerrit-HasComments: No


[kudu-CR](gh-pages) [site] add more docs for Kudu 1.12.0 release

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15945 )

Change subject: [site] add more docs for Kudu 1.12.0 release
..

[site] add more docs for Kudu 1.12.0 release

Change-Id: If7fa629219c77907d49371d1754103d6286677a2
Reviewed-on: http://gerrit.cloudera.org:8080/15945
Reviewed-by: Hao Hao 
Tested-by: Hao Hao 
---
M releases/1.12.0/cpp-client-api/annotated.html
M releases/1.12.0/cpp-client-api/callbacks_8h_source.html
M releases/1.12.0/cpp-client-api/classKuduPartialRow-members.html
M releases/1.12.0/cpp-client-api/classKuduPartialRow.html
M releases/1.12.0/cpp-client-api/classes.html
M releases/1.12.0/cpp-client-api/classkudu_1_1MonoDelta-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1MonoDelta.html
M releases/1.12.0/cpp-client-api/classkudu_1_1MonoTime-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1MonoTime.html
M releases/1.12.0/cpp-client-api/classkudu_1_1Slice-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1Slice.html
M releases/1.12.0/cpp-client-api/classkudu_1_1Status-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1Status.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduBloomFilter-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduBloomFilter.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduBloomFilterBuilder-members.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduBloomFilterBuilder.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduClient-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduClient.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduClientBuilder-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduClientBuilder.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnSchema-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnSchema.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnSpec-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnSpec.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnStorageAttributes-members.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnStorageAttributes.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnTypeAttributes-members.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnTypeAttributes.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnarScanBatch-members.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduColumnarScanBatch.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduDelete-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduDelete.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduError-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduError.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduInsert-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduInsert.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduInsertIgnore-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduInsertIgnore.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduLoggingCallback-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduLoggingCallback.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduLoggingFunctionCallback-members.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduLoggingFunctionCallback.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduLoggingMemberCallback-members.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduLoggingMemberCallback.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduPartitioner-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduPartitioner.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduPartitionerBuilder-members.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduPartitionerBuilder.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduPredicate-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduPredicate.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduReplica-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduReplica.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduScanBatch-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduScanBatch.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduScanToken-members.html
M releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduScanToken.html
M 
releases/1.12.0/cpp-client-api/classkudu_1_1client_1_1KuduScanTokenBuilder-members.html
M 

[kudu-CR] KUDU-2817: Upgrade libev to 4.33

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15937 )

Change subject: KUDU-2817: Upgrade libev to 4.33
..


Patch Set 1:

They are definitely related. I will look into it when I have time.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic171fd8287c3be889e06aa8708cbfbb2aa5e6b45
Gerrit-Change-Number: 15937
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 21:11:30 +
Gerrit-HasComments: No


[kudu-CR] KUDU-2817: Upgrade libev to 4.33

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15937 )

Change subject: KUDU-2817: Upgrade libev to 4.33
..


Patch Set 1:

I didn't look deep enough and I'm not sure whether the failures 
TServerQuiescingParamITest.TestScansRetry scenario are directly related to this 
patch, but it looks suspicious.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic171fd8287c3be889e06aa8708cbfbb2aa5e6b45
Gerrit-Change-Number: 15937
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 21:01:19 +
Gerrit-HasComments: No


[kudu-CR] KUDU-2817: Upgrade snappy to 1.1.8

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15927 )

Change subject: KUDU-2817: Upgrade snappy to 1.1.8
..

KUDU-2817: Upgrade snappy to 1.1.8

We are currently on snappy 1.1.4 which is from 2017 and the latest
release is from January 2020.

This patch upgrades to snappy 1.1.8 which pulls in various fixes and
improvements. See the release notes for all the changes:
https://github.com/google/snappy/releases

I adjusted the build scripts now that snappy uses cmake instead of
autoconf. I used the parameters passed to lz4 for inspiration.

Change-Id: I19e4eda122c0ea957bbe504fc7de681bb72068cd
Reviewed-on: http://gerrit.cloudera.org:8080/15927
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M thirdparty/build-definitions.sh
M thirdparty/download-thirdparty.sh
M thirdparty/vars.sh
3 files changed, 11 insertions(+), 11 deletions(-)

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

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I19e4eda122c0ea957bbe504fc7de681bb72068cd
Gerrit-Change-Number: 15927
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] KUDU-2817: Upgrade bison to 3.5.4

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15935 )

Change subject: KUDU-2817: Upgrade bison to 3.5.4
..

KUDU-2817: Upgrade bison to 3.5.4

This patch upgrades bison from 3.0.5 to 3.5.4.

This is a follow on change to https://gerrit.cloudera.org/#/c/13446

Change-Id: I768d1a59d68b393f7a465a7aba8a4d9e6b22ce11
Reviewed-on: http://gerrit.cloudera.org:8080/15935
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
---
M thirdparty/vars.sh
1 file changed, 1 insertion(+), 1 deletion(-)

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

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I768d1a59d68b393f7a465a7aba8a4d9e6b22ce11
Gerrit-Change-Number: 15935
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] KUDU-2817: Upgrade snappy to 1.1.8

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15927 )

Change subject: KUDU-2817: Upgrade snappy to 1.1.8
..


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15927/2/thirdparty/build-definitions.sh
File thirdparty/build-definitions.sh:

http://gerrit.cloudera.org:8080/#/c/15927/2/thirdparty/build-definitions.sh@528
PS2, Line 528: -DSNAPPY_BUILD_TESTS=OFF
> Just curious: whether it's because it's not compiling or this is to save so
I saw a TSAN instrumentation issue in the test code, but ideally we should 
disable building tests in thirdparty anyway.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I19e4eda122c0ea957bbe504fc7de681bb72068cd
Gerrit-Change-Number: 15927
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 21:00:37 +
Gerrit-HasComments: Yes


[kudu-CR](branch-1.12.x) docs: update the steps to update directories

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15941 )

Change subject: docs: update the steps to update directories
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I3b5f8b6ca548dd34cc866c338ca3b233da472e11
Gerrit-Change-Number: 15941
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 20:45:04 +
Gerrit-HasComments: No


[kudu-CR] KUDU-2817: Upgrade bison to 3.5.4

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15935 )

Change subject: KUDU-2817: Upgrade bison to 3.5.4
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I768d1a59d68b393f7a465a7aba8a4d9e6b22ce11
Gerrit-Change-Number: 15935
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 19:41:56 +
Gerrit-HasComments: No


[kudu-CR](branch-1.12.x) docs: add docs to orchestrate a rolling restart

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15943 )

Change subject: docs: add docs to orchestrate a rolling restart
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I268928ccdf23863880349716b9e5a848a0e443bb
Gerrit-Change-Number: 15943
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 19:10:19 +
Gerrit-HasComments: No


[kudu-CR](branch-1.12.x) docs: add docs to orchestrate a rolling restart

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15943 )

Change subject: docs: add docs to orchestrate a rolling restart
..

docs: add docs to orchestrate a rolling restart

Change-Id: I268928ccdf23863880349716b9e5a848a0e443bb
Reviewed-on: http://gerrit.cloudera.org:8080/15930
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Grant Henke 
(cherry picked from commit 161dec90a10aa96fcc1d2ad789743f6bb37e0d48)
Reviewed-on: http://gerrit.cloudera.org:8080/15943
Reviewed-by: Hao Hao 
---
M docs/administration.adoc
1 file changed, 35 insertions(+), 2 deletions(-)

Approvals:
  Hao Hao: Looks good to me, approved
  Kudu Jenkins: Verified

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I268928ccdf23863880349716b9e5a848a0e443bb
Gerrit-Change-Number: 15943
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.12.x) docs: add docs to orchestrate a rolling restart

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15943


Change subject: docs: add docs to orchestrate a rolling restart
..

docs: add docs to orchestrate a rolling restart

Change-Id: I268928ccdf23863880349716b9e5a848a0e443bb
Reviewed-on: http://gerrit.cloudera.org:8080/15930
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Grant Henke 
(cherry picked from commit 161dec90a10aa96fcc1d2ad789743f6bb37e0d48)
---
M docs/administration.adoc
1 file changed, 35 insertions(+), 2 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I268928ccdf23863880349716b9e5a848a0e443bb
Gerrit-Change-Number: 15943
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 


[kudu-CR](branch-1.12.x) docs: mention how to decommission a tablet server using the rebalancer

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15942 )

Change subject: docs: mention how to decommission a tablet server using the 
rebalancer
..

docs: mention how to decommission a tablet server using the rebalancer

Change-Id: I6f90eb352c4dce6d200cec33dde91c4bdb4aa522
Reviewed-on: http://gerrit.cloudera.org:8080/15929
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Grant Henke 
(cherry picked from commit 3b60ddf1a3f039647db589aa641995cc81dd6857)
Reviewed-on: http://gerrit.cloudera.org:8080/15942
Reviewed-by: Hao Hao 
---
M docs/administration.adoc
1 file changed, 26 insertions(+), 7 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Hao Hao: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I6f90eb352c4dce6d200cec33dde91c4bdb4aa522
Gerrit-Change-Number: 15942
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.12.x) docs: mention how to decommission a tablet server using the rebalancer

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15942 )

Change subject: docs: mention how to decommission a tablet server using the 
rebalancer
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I6f90eb352c4dce6d200cec33dde91c4bdb4aa522
Gerrit-Change-Number: 15942
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 19:09:48 +
Gerrit-HasComments: No


[kudu-CR](branch-1.12.x) docs: update the steps to update directories

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15941 )

Change subject: docs: update the steps to update directories
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I3b5f8b6ca548dd34cc866c338ca3b233da472e11
Gerrit-Change-Number: 15941
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 19:09:33 +
Gerrit-HasComments: No


[kudu-CR](branch-1.12.x) docs: update the steps to update directories

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15941 )

Change subject: docs: update the steps to update directories
..

docs: update the steps to update directories

The `kudu fs update_dirs` tool is no longer required to update the set
of data directories.

Change-Id: I3b5f8b6ca548dd34cc866c338ca3b233da472e11
Reviewed-on: http://gerrit.cloudera.org:8080/15928
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Grant Henke 
(cherry picked from commit 90450e47e75ace558a04c98e19f2089d21d71c31)
Reviewed-on: http://gerrit.cloudera.org:8080/15941
Reviewed-by: Hao Hao 
---
M docs/administration.adoc
1 file changed, 40 insertions(+), 25 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Hao Hao: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I3b5f8b6ca548dd34cc866c338ca3b233da472e11
Gerrit-Change-Number: 15941
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR](branch-1.12.x) docs: mention how to decommission a tablet server using the rebalancer

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15942


Change subject: docs: mention how to decommission a tablet server using the 
rebalancer
..

docs: mention how to decommission a tablet server using the rebalancer

Change-Id: I6f90eb352c4dce6d200cec33dde91c4bdb4aa522
Reviewed-on: http://gerrit.cloudera.org:8080/15929
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Grant Henke 
(cherry picked from commit 3b60ddf1a3f039647db589aa641995cc81dd6857)
---
M docs/administration.adoc
1 file changed, 26 insertions(+), 7 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6f90eb352c4dce6d200cec33dde91c4bdb4aa522
Gerrit-Change-Number: 15942
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 


[kudu-CR](branch-1.12.x) docs: update the steps to update directories

2020-05-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15941


Change subject: docs: update the steps to update directories
..

docs: update the steps to update directories

The `kudu fs update_dirs` tool is no longer required to update the set
of data directories.

Change-Id: I3b5f8b6ca548dd34cc866c338ca3b233da472e11
Reviewed-on: http://gerrit.cloudera.org:8080/15928
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin 
Reviewed-by: Grant Henke 
(cherry picked from commit 90450e47e75ace558a04c98e19f2089d21d71c31)
---
M docs/administration.adoc
1 file changed, 40 insertions(+), 25 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3b5f8b6ca548dd34cc866c338ca3b233da472e11
Gerrit-Change-Number: 15941
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Andrew Wong 


[kudu-CR] KUDU-2817: Upgrade snappy to 1.1.8

2020-05-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15927 )

Change subject: KUDU-2817: Upgrade snappy to 1.1.8
..


Patch Set 2: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15927/2/thirdparty/build-definitions.sh
File thirdparty/build-definitions.sh:

http://gerrit.cloudera.org:8080/#/c/15927/2/thirdparty/build-definitions.sh@528
PS2, Line 528: -DSNAPPY_BUILD_TESTS=OFF
Just curious: whether it's because it's not compiling or this is to save some 
time on compilation?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I19e4eda122c0ea957bbe504fc7de681bb72068cd
Gerrit-Change-Number: 15927
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Bankim Bhavsar 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 18:30:36 +
Gerrit-HasComments: Yes


[kudu-CR] [KUDU-3116] Enhance KuduContext row operation metrics

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15882 )

Change subject: [KUDU-3116] Enhance KuduContext row operation metrics
..


Patch Set 8: Code-Review+1


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie66dab95041310c27ef62dacccbcc0977a84857e
Gerrit-Change-Number: 15882
Gerrit-PatchSet: 8
Gerrit-Owner: Brian McDevitt 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Brian McDevitt 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 18 May 2020 14:46:50 +
Gerrit-HasComments: No


[kudu-CR] KUDU-3096: Upgrade libunwind to 1.4.0

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15899 )

Change subject: KUDU-3096: Upgrade libunwind to 1.4.0
..

KUDU-3096: Upgrade libunwind to 1.4.0

libunwind 1.3.1 use malloc in thread stack trace collection,
that cause core dump and potential deadlock between GetThreadStack
and SuperviseThread, GetThreadStack function will try to collect
thread info even if tcmalloc ThreadCache haven't been inited completlly,
the issue happen in ARM64 server. In libunwind 1.4.0, upstream have
fixed malloc issue, so upgrade it.

Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
Reviewed-on: http://gerrit.cloudera.org:8080/15899
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke 
---
M thirdparty/vars.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Grant Henke: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
Gerrit-Change-Number: 15899
Gerrit-PatchSet: 4
Gerrit-Owner: RuiChen 
Gerrit-Reviewer: Anonymous Coward 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: RuiChen 
Gerrit-Reviewer: Todd Lipcon 


[kudu-CR] KUDU-3096: Upgrade libunwind to 1.4.0

2020-05-18 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15899 )

Change subject: KUDU-3096: Upgrade libunwind to 1.4.0
..


Patch Set 3: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
Gerrit-Change-Number: 15899
Gerrit-PatchSet: 3
Gerrit-Owner: RuiChen 
Gerrit-Reviewer: Anonymous Coward 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: RuiChen 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Mon, 18 May 2020 12:48:31 +
Gerrit-HasComments: No


[kudu-CR] KUDU-3096: Upgrade libunwind to 1.4.0

2020-05-18 Thread RuiChen (Code Review)
RuiChen has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15899 )

Change subject: KUDU-3096: Upgrade libunwind to 1.4.0
..


Patch Set 3:

> Hi all,
 > I have took test after upgrade libunwind to 1.4.0, it works, so I
 > think we only have to upgrade libunwind for this issue.

Yes, I upgrade libunwind to 1.4.0 and run test case 
DebugUtilTest.TestThreadBlockingSignals 1000 times, all tests passed. So I 
think upgrading worked for ARM64.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
Gerrit-Change-Number: 15899
Gerrit-PatchSet: 3
Gerrit-Owner: RuiChen 
Gerrit-Reviewer: Anonymous Coward 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: RuiChen 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Mon, 18 May 2020 11:41:15 +
Gerrit-HasComments: No


[kudu-CR] KUDU-3096: Upgrade libunwind to 1.4.0

2020-05-18 Thread RuiChen (Code Review)
Hello Kudu Jenkins, huangtianhua...@gmail.com, Grant Henke, Todd Lipcon, 

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

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

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

Change subject: KUDU-3096: Upgrade libunwind to 1.4.0
..

KUDU-3096: Upgrade libunwind to 1.4.0

libunwind 1.3.1 use malloc in thread stack trace collection,
that cause core dump and potential deadlock between GetThreadStack
and SuperviseThread, GetThreadStack function will try to collect
thread info even if tcmalloc ThreadCache haven't been inited completlly,
the issue happen in ARM64 server. In libunwind 1.4.0, upstream have
fixed malloc issue, so upgrade it.

Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
---
M thirdparty/vars.sh
1 file changed, 1 insertion(+), 1 deletion(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
Gerrit-Change-Number: 15899
Gerrit-PatchSet: 3
Gerrit-Owner: RuiChen 
Gerrit-Reviewer: Anonymous Coward 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: RuiChen 
Gerrit-Reviewer: Todd Lipcon 


[kudu-CR] KUDU-3096: Fix core dump when thread race between GetThreadStack and SuperviseThread

2020-05-18 Thread Anonymous Coward (Code Review)
huangtianhua...@gmail.com has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15899 )

Change subject: KUDU-3096: Fix core dump when thread race between 
GetThreadStack and SuperviseThread
..


Patch Set 2:

Hi all,
I have took test after upgrade libunwind to 1.4.0, it works, so I think we only 
have to upgrade libunwind for this issue.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
Gerrit-Change-Number: 15899
Gerrit-PatchSet: 2
Gerrit-Owner: RuiChen 
Gerrit-Reviewer: Anonymous Coward 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: RuiChen 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Mon, 18 May 2020 07:34:11 +
Gerrit-HasComments: No