Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor. [build-issue] Make sure to install setuptools less then version 36.0.0 to avoid module six not found issue #341

Merged
merged 1 commit into from
Jun 2, 2017

Conversation

jerryshao
Copy link
Contributor

The issue is tracked here pypa/setuptools#1042.

…ule six not found issue

Change-Id: I7ba64be56354be5a03e6f6b349990af6b662e457
@jerryshao
Copy link
Contributor Author

CC @zjffdu , please review. This blocks our travis building.

@codecov-io
Copy link

Codecov Report

Merging #341 into master will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #341      +/-   ##
===========================================
+ Coverage     70.46%   70.5%   +0.03%     
- Complexity      726     727       +1     
===========================================
  Files            96      96              
  Lines          5123    5123              
  Branches        774     774              
===========================================
+ Hits           3610    3612       +2     
+ Misses          996     995       -1     
+ Partials        517     516       -1
Impacted Files Coverage Δ Complexity Δ
...va/com/cloudera/livy/rsc/rpc/KryoMessageCodec.java 98.11% <0%> (+3.77%) 19% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2abb8a3...e453908. Read the comment docs.

@zjffdu
Copy link
Contributor

zjffdu commented Jun 2, 2017

LGTM

@zjffdu zjffdu merged commit 02eef9a into cloudera:master Jun 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants