Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vdk-core: relevant info in step result #3062

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

DeltaMichael
Copy link
Contributor

@DeltaMichael DeltaMichael commented Jan 29, 2024

Why?

Sometimes the exact errors in user code are hard to find in the stack trace

What?

Print relevant information from stack trace in step results

Example:

On error in step 20_python_step.py, the short result summary looks like this

2024-01-29 15:14:14,736 [VDK] hello-world [INFO ] vdk.internal.builtin_plugins.r           cli_run.py:166  __log_short_exec[id:3a22ec2e-d6e6-4f94-96c0-122955930cbb-1706534052-abffd]- Job execution result: ERROR
Step results:
10_sql_step.sql - SUCCESS
20_python_step.py - ERROR
  File "/Users/mdilyan/Projects/vdk-playground/hello-world/20_python_step.py", line 27, in run
    raise ValueError("No value in this")
ValueError: No value in this

How was this tested?

Ran locally
Functional test

What kind of change is this?

Feature/non-breaking

@DeltaMichael DeltaMichael force-pushed the person/mdilyan/relevant-info-step branch 2 times, most recently from f8f358f to 641621b Compare January 30, 2024 12:56
Why?

Sometimes the exact errors in user code are hard
to find in the stack trace

What?

Print relevant information from stack trace in
step results

How was this tested?

Ran locally
Functional test

What kind of change is this?

Feature/non-breaking

Signed-off-by: Dilyan Marinov <[email protected]>
@DeltaMichael DeltaMichael force-pushed the person/mdilyan/relevant-info-step branch from 641621b to c1e1b84 Compare January 30, 2024 12:57
Copy link
Collaborator

@duyguHsnHsn duyguHsnHsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me

@DeltaMichael DeltaMichael merged commit 656ba0c into main Feb 1, 2024
10 checks passed
@DeltaMichael DeltaMichael deleted the person/mdilyan/relevant-info-step branch February 1, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants