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

Add athena plugin #272

Merged
merged 2 commits into from
Jun 9, 2021
Merged

Add athena plugin #272

merged 2 commits into from
Jun 9, 2021

Conversation

katrogan
Copy link
Contributor

@katrogan katrogan commented Jun 2, 2021

Signed-off-by: Katrina Rogan [email protected]

Read then delete

  • Make sure to use a concise title for the pull-request.
  • Use #patch, #minor or #major in the pull-request title to bump the corresponding version. Otherwise, the patch version
    will be bumped. More details

TL;DR

Please replace this text with a description of what this PR accomplishes.

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

https://github.com/flyteorg/flyte/issues/

Follow-up issue

NA
OR
https://github.com/flyteorg/flyte/issues/

Signed-off-by: Katrina Rogan <[email protected]>
@katrogan katrogan requested review from EngHabu and kumare3 as code owners June 2, 2021 23:37
Signed-off-by: Katrina Rogan <[email protected]>
@codecov
Copy link

codecov bot commented Jun 2, 2021

Codecov Report

Merging #272 (3d7158e) into master (0437b71) will not change coverage.
The diff coverage is n/a.

@kumare3
Copy link
Contributor

kumare3 commented Jun 9, 2021

@katrogan - @EngHabu has a previous PR for this, should we merge this?

@katrogan
Copy link
Contributor Author

katrogan commented Jun 9, 2021

@kumare3 yes, I believe @EngHabu closed that PR (and it was quite old, so not worth resurrecting b/c of merge conflicts)

@katrogan katrogan merged commit 5ce5085 into master Jun 9, 2021
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
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