Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Enable Presto plugin #96

Closed
wants to merge 2 commits into from
Closed

Enable Presto plugin #96

wants to merge 2 commits into from

Conversation

lu4nm3
Copy link

@lu4nm3 lu4nm3 commented Mar 20, 2020

TL;DR

This enables the use of the Presto plugin

Type

  • Bug Fix
  • Feature
  • Plugin

Tracking Issue

flyteorg/flyte#203

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
+ Coverage   49.73%   49.77%   +0.03%     
==========================================
  Files         127      127              
  Lines        7928     7948      +20     
==========================================
+ Hits         3943     3956      +13     
- Misses       3601     3608       +7     
  Partials      384      384              
Impacted Files Coverage Δ
pkg/controller/nodes/executor.go 58.54% <0.00%> (-0.30%) ⬇️
pkg/apis/flyteworkflow/v1alpha1/node_status.go 17.56% <0.00%> (-0.10%) ⬇️
pkg/controller/nodes/dynamic/handler.go 63.78% <0.00%> (+1.39%) ⬆️

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 e90c9e2...90cc2c0. Read the comment docs.

@kumare3 kumare3 closed this May 29, 2020
@kumare3 kumare3 deleted the lu4nm3/presto branch May 29, 2020 17:39
kumare3 pushed a commit to nuclyde-io/flytepropeller that referenced this pull request Feb 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants