feat(ingestion): Add Pinecone Vector DB Source with skill_docs #datahub-skills#16472
Open
jishanahmed-shaikh wants to merge 42 commits intodatahub-project:masterfrom
Open
feat(ingestion): Add Pinecone Vector DB Source with skill_docs #datahub-skills#16472jishanahmed-shaikh wants to merge 42 commits intodatahub-project:masterfrom
jishanahmed-shaikh wants to merge 42 commits intodatahub-project:masterfrom
Conversation
…ocs #datahub-skills
github-actions
bot
added
ingestion
maggiehays
added
the
needs-review
- Add capability decorators for PLATFORM_INSTANCE, DOMAINS, CONTAINERS - Remove unused imports (ContainerClass, ContainerPropertiesClass, Dict) - Fix F541: Remove unnecessary f-string prefix in pinecone_client.py - Fix SIM114: Combine isinstance checks in schema_inference.py - Remove unused variable assignments in test_pinecone_source.py - Format markdown documentation with prettier - Register Pinecone in connector registry with all 5 capabilities Fixes PR datahub-project#16472
…ation formatting - Add missing 'Dict' import to pinecone_source.py (fixes NameError) - Remove unused 'Dict' import from report.py - Re-sort import blocks in test_pinecone_source.py - Apply final Prettier formatting to skill_docs/*.md All local ruff checks passed.
Contributor
|
What about adding some integration tests with Pinecone emulator available as a Docker image? https://docs.pinecone.io/guides/operations/local-development |
Author
Sounds Good, giving it a try |
…afety (datahub-project#16444) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…nts (datahub-project#16452) Co-authored-by: John Joyce <john@ip-192-168-1-212.us-west-2.compute.internal> Co-authored-by: John Joyce <john@Mac-3236.lan>
Co-authored-by: John Joyce <john@ip-192-168-1-212.us-west-2.compute.internal> Co-authored-by: John Joyce <john@Mac-3094.lan>
…datahub-project#16476) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…ausing 422 on transaction retry (datahub-project#16474)
Co-authored-by: John Joyce <john@Mac-3608.lan>
Bundle ReportChanges will increase total bundle size by 27 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: datahub-react-web-esmAssets Changed:
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
…ng pinecone module
…ndency from integration test
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR introduces a production-grade Pinecone ingestion source, developed and verified using the datahub-skills framework with a 9.9/10 review score.
🚀 Key Features:
#datahub-skills