You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Adar Dembo (Code Review)" <ge...@cloudera.org> on 2017/09/29 17:31:22 UTC

[kudu-CR](branch-1.5.x) python: pin to cython 0.26.1

Adar Dembo has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8174


Change subject: python: pin to cython 0.26.1
......................................................................

python: pin to cython 0.26.1

Some of our el6-based builds are running into an installation issue [1] that
appears to have surfaced in Cython 0.27. We don't actually need any features
from recent Cython releases, so let's pin to the last one known to be good.

1. https://github.com/cython/cython/issues/1879

Change-Id: I201f4c078ff1308f267f73275f7cacb35161a2d3
Reviewed-on: http://gerrit.cloudera.org:8080/8155
Reviewed-by: Jean-Daniel Cryans <jd...@apache.org>
Tested-by: Kudu Jenkins
(cherry picked from commit 36f4259f425d2e756153863deb83ca40478331cd)
---
M python/requirements.txt
1 file changed, 1 insertion(+), 1 deletion(-)



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.5.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I201f4c078ff1308f267f73275f7cacb35161a2d3
Gerrit-Change-Number: 8174
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>

[kudu-CR](branch-1.5.x) python: pin to cython 0.26.1

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

Change subject: python: pin to cython 0.26.1
......................................................................

python: pin to cython 0.26.1

Some of our el6-based builds are running into an installation issue [1] that
appears to have surfaced in Cython 0.27. We don't actually need any features
from recent Cython releases, so let's pin to the last one known to be good.

1. https://github.com/cython/cython/issues/1879

Change-Id: I201f4c078ff1308f267f73275f7cacb35161a2d3
Reviewed-on: http://gerrit.cloudera.org:8080/8155
Reviewed-by: Jean-Daniel Cryans <jd...@apache.org>
Tested-by: Kudu Jenkins
(cherry picked from commit 36f4259f425d2e756153863deb83ca40478331cd)
Reviewed-on: http://gerrit.cloudera.org:8080/8174
Reviewed-by: Todd Lipcon <to...@apache.org>
---
M python/requirements.txt
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Todd Lipcon: Looks good to me, approved
  Kudu Jenkins: Verified

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.5.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I201f4c078ff1308f267f73275f7cacb35161a2d3
Gerrit-Change-Number: 8174
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>

[kudu-CR](branch-1.5.x) python: pin to cython 0.26.1

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

Change subject: python: pin to cython 0.26.1
......................................................................


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.5.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I201f4c078ff1308f267f73275f7cacb35161a2d3
Gerrit-Change-Number: 8174
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Comment-Date: Fri, 29 Sep 2017 17:34:11 +0000
Gerrit-HasComments: No