<#6652 [FEAT] Add function name to TaskMetadata pr...
# flyte-github
a
#6652 [FEAT] Add function name to TaskMetadata proto Pull request opened by machichima ## Tracking issue ## Why are the changes needed? Short name is not always equal to function name. In the following cases they will be different: # function name: testtask / short name: rename @env.task(short_name="rename") async def testtask(data: str = "default string", n: int = 3) -> str: return data Therefor we should have a separate field for
function_name
## What changes were proposed in this pull request? Add
function_name
field into
TaskMetadata
Based on the offline discussion with @pingsutw , we decided to rename
short_name
as
function_name
instead of adding a new field ## How was this patch tested? ### Labels Please add one or more of the following labels to categorize your PR: • added: For new features. • changed: For changes in existing functionality. • deprecated: For soon-to-be-removed features. • removed: For features being removed. • fixed: For any bug fixed. • security: In case of vulnerabilities This is important to improve the readability of release notes. ### Setup process ### Screenshots ## Check all the applicable boxes • I updated the documentation accordingly. • All new and existing tests passed. • All commits are signed-off. ## Related PRs ## Docs link flyteorg/flyte