feat(construct): merged amazon aurora vector stores
Fixes #373
- Merged
AmazonAuroraDefaultVectorStoreintoAmazonAuroraVectorStoreresulting in a single construct that customer needs to import. - Created
fromExistingAuroraVectorStoreinAmazonAuroraVectorStoreconstruct so customers could import values from an existing Aurora database. - Refactored
modelsand moved it outside ofbedrockconstruct in a dedicatedfoudnationsmodelsconstruct. This results in elimination of potential cross-reference issues in the future where a knowledge base vector is consuming models but is also being referenced withinbedrockconstruct. - Fix an issue where aurora cannot be deployed from Mac
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 93.37%. Comparing base (
8a81eb2) to head (207e39b). Report is 30 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #419 +/- ##
==========================================
+ Coverage 93.03% 93.37% +0.34%
==========================================
Files 47 47
Lines 11183 11338 +155
Branches 359 364 +5
==========================================
+ Hits 10404 10587 +183
+ Misses 779 751 -28
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What's blocking the release of this? I want to get started with this construct library, but don't want to deal with fixing breaking changes later, and after #346 (thanks, 🎉) was patiently waiting for this one too to finally get my hands dirty. 😁
This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label.
Closing this pull request as it hasn't seen activity for a while. Please add a comment @mentioning a maintainer to reopen. If you wish to exclude this issue from being marked as stale, add the "backlog" label.