Skip to content

BSQIK/Search: Indexing #4422

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 150 commits into from
Jul 15, 2025
Merged

BSQIK/Search: Indexing #4422

merged 150 commits into from
Jul 15, 2025

Conversation

sir-sigurd
Copy link
Member

@sir-sigurd sir-sigurd commented Jun 30, 2025 •
edited
Loading

Description

TODO

sir-sigurd and others added 30 commits June 4, 2025 14:23
@sir-sigurd
index pkg entries
5189190
@sir-sigurd
CI
9792c46
@sir-sigurd
lint
d6bc135
@sir-sigurd
fix decorator
fb0822c
@sir-sigurd
use non-deprecated name
ef11bcb
@sir-sigurd
do not index dir entries & add debugging
bd11a04
@sir-sigurd
use prefix for pkg fields, parse pk
25d6b5a
@sir-sigurd
use hash for ids
69666d4
@sir-sigurd
pointers as child docs, is_packaged field
9ce314a
@sir-sigurd
fix pkg_stats name
6136100
@sir-sigurd
change fields names, fixes
99dd238
@sir-sigurd
fix
ec35e53
@sir-sigurd
fixes
ef2b90f
@sir-sigurd
ignore manifests with invalid hashes
6708669
@sir-sigurd
fix total_files
941b664
@nl0
attempt to fix parent ids
28e42d9
@sir-sigurd
fix check
bc0a58a
@sir-sigurd
do not store more than CHUNK_LIMIT_DOCS
b6cb8ef
@sir-sigurd
log errors
5354113
@sir-sigurd
small adjustments
4060819
@sir-sigurd
try http_compress
49616db
@sir-sigurd
disable http_compress, increase CHUNK_LIMIT_DOCS
3f02276
@sir-sigurd
fix, increase CHUNK_LIMIT_DOCS
9d3de8e
@sir-sigurd
increase CHUNK_LIMIT_DOCS
561aa46
@sir-sigurd
change log level
375b1eb
@sir-sigurd
rework indexing
3a16832
@sir-sigurd
fix
c065497
@sir-sigurd
fix
0fc4c33
@sir-sigurd
fix
0368b10
@sir-sigurd
fix
acee91b
@sir-sigurd sir-sigurd mentioned this pull request Jul 9, 2025
Merged
1 task
@sir-sigurd sir-sigurd requested a review from nl0 July 14, 2025 11:51
nl0
nl0 previously approved these changes Jul 15, 2025
View reviewed changes
Copy link
Member

@nl0 nl0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tho it would be nice to have more type coverage.
do you plan on addressing the rest of the TODO/XXX comments?

@sir-sigurd
Revert "revert CI"
e8731ac 
This reverts commit 9801de7.
@sir-sigurd
Reapply "revert CI"
697fd2e 
This reverts commit e8731ac.
@sir-sigurd sir-sigurd requested a review from nl0 July 15, 2025 12:52
@sir-sigurd sir-sigurd enabled auto-merge July 15, 2025 13:17
@sir-sigurd sir-sigurd added this pull request to the merge queue Jul 15, 2025
Merged via the queue into master with commit e0ae23a Jul 15, 2025
38 of 39 checks passed
@sir-sigurd sir-sigurd deleted the index-pkg-entries-2 branch July 15, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@nl0 nl0 nl0 approved these changes

@kevinemoore kevinemoore Awaiting requested review from kevinemoore

+1 more reviewer

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sir-sigurd @drernie @nl0