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

feat; catalog api update to return additional information for catalog #105

Merged
merged 7 commits into from
Jul 28, 2020

Conversation

kumare3
Copy link
Contributor

@kumare3 kumare3 commented Jul 24, 2020

TL;DR

CacheStatus & Catalog Metadata is returned as part of Catalog Addition and Catalog retrieval

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

Prior to this PR, Catalog GET/PUT would only retrieve status of the PUT and in case of GET return previous outputs. This change makes it possible to retrieve associated metadata about the GET/PUT so that we can actually create the lineage information

Tracking Issue

flyteorg/flyte#138

Follow-up issue

NA

@codecov-commenter
Copy link

codecov-commenter commented Jul 24, 2020

Codecov Report

Merging #105 into master will decrease coverage by 0.22%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
- Coverage   59.74%   59.52%   -0.23%     
==========================================
  Files          95       95              
  Lines        5334     5354      +20     
==========================================
  Hits         3187     3187              
- Misses       1845     1865      +20     
  Partials      302      302              
Impacted Files Coverage Δ
go/tasks/pluginmachinery/catalog/client.go 10.52% <0.00%> (-29.48%) ⬇️
.../tasks/pluginmachinery/catalog/reader_processor.go 16.66% <0.00%> (-0.99%) ⬇️
.../tasks/pluginmachinery/catalog/writer_processor.go 20.83% <0.00%> (-4.17%) ⬇️

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 e4ee895...8bd7990. Read the comment docs.

@kumare3 kumare3 merged commit 9ac6eeb into master Jul 28, 2020
eapolinario pushed a commit that referenced this pull request Sep 6, 2023
…#105)

* Updating Core catalog api to return more information to the engine

* use latest

* Go modules update

* lint fixed

* FlyteIDL updated v0.17.38
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