Add deck_uri to NodeExecutionEvent

[pingsutw]

Signed-off-by: Kevin Su pingsutw@apache.org

TL;DR

Add deck_uri in NodeExecutionEvent

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

flyteorg/flyte#2175

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

Follow-up issue

NA

[codecov]

Codecov Report

Merging #443 (443785f) into master (d5ba4c7) will increase coverage by 0.03%.
The diff coverage is 75.00%.

[pingsutw]

cc @kumare3 @EngHabu

[EngHabu]

Thank you, @pingsutw I think this is close!

[pingsutw]

Few updates, cc @EngHabu

• Upgraded flyteplugin
• Drop the changes in datacatalog (will create a new issue for caching deck)
• Add deck_uri in outputInfo

[pingsutw]

cc @EngHabu

[hamersaw]

One small question. I've tried to shy away from adding more to the NodeStatus on the NodeExecutionContext because it "claims" to be deprecated from the NodeExecutionContext. If this is the case we may want to find a better location for the deckURI; otherwise, it may be nice to remove the deprecated comment.

[pingsutw]

I think it's ok to remove NodeStatus. IIUC, we will replace NodeStatus with OutputWriter. if so, we can use OutputWriter.GetDeckPath() to get deck_uri.

[pingsutw]

Fixed it #453