-
Notifications
You must be signed in to change notification settings - Fork 3
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: parent retriever/indexer #16
base: main
Are you sure you want to change the base?
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.
license-eye has totally checked 152 files.
Valid | Invalid | Ignored | Fixed |
---|---|---|---|
148 | 4 | 0 | 0 |
Click to see the invalid file list
- flow/indexer/parent/parent.go
- flow/indexer/parent/parent_test.go
- flow/retriever/parent/parent.go
- flow/retriever/parent/parent_test.go
9395eef
to
e6b0a9e
Compare
go-test-coverage report:
|
1eb08b1
to
cc39e05
Compare
e6b0a9e
to
7ad2207
Compare
|
||
if subDoc.ID == currentID { | ||
continue | ||
} |
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.
这段没理解
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.
有点问题,改了
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 == 0 的时候,一定会 continue?
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.
对,等到id发生变化之后更新之前的docid
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.
这里是假定了同一个 parent doc 的 sub doc,ID 一定是相同的?
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.
是的
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.
done
471c382
to
8d65eb9
Compare
8d65eb9
to
b39dd0d
Compare
add parent retriever/indexer