You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2019/11/14 19:46:17 UTC

[kudu-CR](branch-1.10.x) thirdparty: remove memkind from LICENSE.txt 

Hello Kudu Jenkins,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: thirdparty: remove memkind from LICENSE.txt
......................................................................

thirdparty: remove memkind from LICENSE.txt

I'm getting kinda overwhelmed by the sheer number of license files we seem
to be juggling. I guess that's how we missed this one.

Change-Id: I7989fca644f573e90304af4cbf5dc2440fe37d3e
Reviewed-on: http://gerrit.cloudera.org:8080/14703
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <as...@cloudera.com>
(cherry picked from commit 7e1bcb63958ad3e672322a005f441e5dd3a9d087)
  Conflicts:
      thirdparty/LICENSE.txt
---
M thirdparty/LICENSE.txt
1 file changed, 1 insertion(+), 61 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.10.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7989fca644f573e90304af4cbf5dc2440fe37d3e
Gerrit-Change-Number: 14708
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR](branch-1.10.x) thirdparty: remove memkind from LICENSE.txt

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14708 )

Change subject: thirdparty: remove memkind from LICENSE.txt
......................................................................


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

It's a useful update to bring the state of LICENSE.txt up-to-date.  Since 1.10.1-RC2 didn't pass, I'm using the opportunity to add this into branch-1.10.x

This is a doc/license-only update, no build verification is needed.


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.10.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I7989fca644f573e90304af4cbf5dc2440fe37d3e
Gerrit-Change-Number: 14708
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 14 Nov 2019 20:02:30 +0000
Gerrit-HasComments: No

[kudu-CR](branch-1.10.x) thirdparty: remove memkind from LICENSE.txt

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14708 )

Change subject: thirdparty: remove memkind from LICENSE.txt
......................................................................

thirdparty: remove memkind from LICENSE.txt

I'm getting kinda overwhelmed by the sheer number of license files we seem
to be juggling. I guess that's how we missed this one.

Change-Id: I7989fca644f573e90304af4cbf5dc2440fe37d3e
Reviewed-on: http://gerrit.cloudera.org:8080/14703
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <as...@cloudera.com>
(cherry picked from commit 7e1bcb63958ad3e672322a005f441e5dd3a9d087)
  Conflicts:
      thirdparty/LICENSE.txt
Reviewed-on: http://gerrit.cloudera.org:8080/14708
Tested-by: Alexey Serbin <as...@cloudera.com>
---
M thirdparty/LICENSE.txt
1 file changed, 1 insertion(+), 61 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved; Verified

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.10.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I7989fca644f573e90304af4cbf5dc2440fe37d3e
Gerrit-Change-Number: 14708
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)