-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
fix: postgres-adapter - remove nonsensical schema check #1377
fix: postgres-adapter - remove nonsensical schema check #1377
Conversation
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.
Thanks
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.
Resolve merge conflict then good to go
a lot of queries on each start up but probably fast ones |
Yah, but i dont think its an issue because were not running on a serverless runtime. eliza has to run on a box. Also idk if eliza is already doing this but all startup IO operations should be parallelized to minimize total startup time. |
07bbd66
to
8e456a2
Compare
Relates to:
Closes: #1376
Risks
Background
What does this PR do?
What kind of change is this?
Documentation changes needed?
Testing
Where should a reviewer start?
Detailed testing steps