qubole / rubix

Cache File System optimized for columnar formats and object stores

Apache License 2.0

183 stars 74 forks source link

Fixed the httpclient version #229

Closed kvankayala closed 5 years ago

kvankayala commented 6 years ago

Addressed the httpclient security alert

codecov-io commented 6 years ago

Codecov Report

Merging #229 into master will not change coverage. The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #229   +/-   ##
======================================
  Coverage    22.4%   22.4%           
======================================
  Files          59      59           
  Lines        2857    2857           
  Branches      307     307           
======================================
  Hits          640     640           
  Misses       2177    2177           
  Partials       40      40

abhishekdas99 commented 5 years ago

@kvankayala Can you please check whether we need this dependency. If not can we remove this ?

kvankayala commented 5 years ago

The Travis tests are failing for some other reason. This change works good and it not causing any travis build failures in https://github.com/qubole/rubix/pull/234/commits/58562032a7d50c41aba8d7d9550722b6e5884ba1 @abhishekdas99 Can you please merge this PR to remove httplibrary ?