Skip to content
This repository has been archived by the owner on Apr 8, 2024. It is now read-only.

feat(fal): offer run_result for models in context #588

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chamini2
Copy link
Member

Description

Add run_result information for a model. We are missing tests.

run result: RunResultOutput(status=<RunStatus.Success: 'success'>, timing=[TimingInfo(name='compile', started_at=datetime.datetime(2022, 10, 17, 21, 16, 31, 52868, tzinfo=tzutc()), completed_at=datetime.datetime(2022, 10, 17, 21, 16, 31, 54862, tzinfo=tzutc())), TimingInfo(name='execute', started_at=datetime.datetime(2022, 10, 17, 21, 16, 31, 55018, tzinfo=tzutc()), completed_at=datetime.datetime(2022, 10, 17, 21, 16, 31, 105600, tzinfo=tzutc()))], thread_id='Thread-1', execution_time=0.05340003967285156, adapter_response={'_message': 'SELECT 1', 'code': 'SELECT', 'rows_affected': 1}, message='SELECT 1', failures=None, unique_id='model.fal_005.some_model')

cc: @cmlad

Integration tests

Adapter to test:

  • postgres

Python version to test:

  • 3.8

@chamini2 chamini2 changed the title feat: offer run_result for models in context feat(fal): offer run_result for models in context Nov 2, 2022
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.

1 participant