Merged
Conversation
RunContext는 공개 API(__init__.py)에 포함되지 않으며 v2 컴포넌트/플로우에서 사용되지 않음. 문서 존재 시 사용자가 공개 API로 오인할 수 있어 삭제.
다중 인자 컴포넌트 비호환, 컨텍스트 소실 문제와 전용 Flow(BaselineNL2SQL 등) 사용을 권장하는 해결 방법 추가
- vector-retriever: IndexBuilder 제거, BaselineNL2SQL retriever= 오류 수정 (HybridNL2SQL로 교체),
add() 시그니처 수정(pre-split 필수), splitter 파라미터명 반영
- v2-usage-guide: EnrichedNL2SQL 추가, embedding 6개/vectorstore 3개로 목록 현행화
- getting-started-without-datahub: langchain import 제거, FAISSVectorStore/PGVectorStore 네이티브 코드로
교체,
pgvector 실행 방법 3가지(Docker, 직접 설치, 클라우드)로 확장
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
1 participant
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.
#️⃣ Issue Number
📝 요약(Summary)
v2 마이그레이션 후속 문서 정리 및 인터페이스 보완. deprecated 문서 제거, 튜토리얼 v2 API 반영,
CatalogLoaderPort 추가 및 DataHubCatalogLoader 계보 문서 로딩 기능 추가.
💬 To Reviewers (선택)
문서 정리
RunContext_ko.md제거 — v2에서 RunContext가 컴포넌트 내부로 완전히 이관되어 별도 문서 불필요.BaseFlow_ko.md에 SequentialFlow 알려진 제한사항 섹션 추가.tutorials/의 IndexBuilder 참조 전면 제거 및from_chunks()/split()패턴으로 교체.CatalogLoaderPort + DataHubCatalogLoader 보완 (
core/ports.py,integrations/catalog/)CatalogLoaderPortProtocol 추가 — 외부 카탈로그 소스를 교체 가능한 인터페이스로 분리.DataHubCatalogLoader에load_lineage_documents()추가 — 계보 정보를 TextDocument로 로드해VectorRetriever에 바로 주입 가능.
PR Checklist
load()동작에 영향 없음 확인reference) How to Code Review