Skip to content

Conversation

@mistercrunch
Copy link
Member

No description provided.

@john-bodley
Copy link
Member

@mistercrunch the thrift reference may be somewhat irrelevant after #7115.

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7117   +/-   ##
=======================================
  Coverage   64.59%   64.59%           
=======================================
  Files         422      422           
  Lines       20587    20587           
  Branches     2251     2251           
=======================================
  Hits        13298    13298           
  Misses       7167     7167           
  Partials      122      122

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 003364e...825e028. Read the comment docs.

@mistercrunch
Copy link
Member Author

Ok, leaving the thrift reference in there just in case it changes something for someone out there (maybe the version ranges that are accepted are different, depending, ...).

@mistercrunch mistercrunch merged commit 87fae5c into apache:master Mar 25, 2019
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.34.0 First shipped in 0.34.0 labels Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.34.0 First shipped in 0.34.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants