-
Notifications
You must be signed in to change notification settings - Fork 1k
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
feat: create base data layer #1592
base: main
Are you sure you want to change the base?
Conversation
ee30b76
to
4dfc1dd
Compare
:type, :start_time, :end_time, :variables, :settings, :tools, | ||
:root_run_id | ||
) | ||
ON CONFLICT (id) DO UPDATE SET |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to go with on conflict
because a step might already exist in DB, just not yet populated.
@@ -79,3 +80,50 @@ async def upload_file( | |||
except Exception as e: | |||
logger.warn(f"AzureStorageClient, upload_file error: {e}") | |||
return {} | |||
|
|||
|
|||
class AzureBlobStorageClient(BaseStorageClient): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Creating a separate Azure client for blobs as it seems more appropriate to cloud storage for attachments.
* chore: clean auth hooks * fix: don't call /user if no auth is configured
Changes:
ChainlitDataLayer
which logs automatically thread/steps/user/feedback to DB with schema aligning with https://github.com/Chainlit/chainlit-datalayerTo test:
I didn't test with Azure.