Skip to content

Conversation

zwpaper
Copy link
Member

@zwpaper zwpaper commented Sep 23, 2025

Note: this is merging into the branch indexing repo in shard.

When job load is high, there are OutOfRange warning in Tabby console:
CleanShot 2025-09-23 at 16 51 43@2x

Since this error is caused by secs < 0:
CleanShot 2025-09-23 at 16 51 58@2x

we can utilize it to check whether the time is past and skip the log

@zwpaper zwpaper requested a review from wsxiaoys September 23, 2025 08:56
@wsxiaoys wsxiaoys merged commit 7e87453 into feat/repo-index-in-shard Sep 24, 2025
2 checks passed
@wsxiaoys wsxiaoys deleted the fix/job-skip-error-duration branch September 24, 2025 05:24
wsxiaoys pushed a commit that referenced this pull request Sep 24, 2025
* feat(index): indexing repo in shard when more than 20

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* chore: add env for enabling index repo in shard

# Conflicts:
#	ee/tabby-webserver/src/service/background_job/mod.rs

* [autofix.ci] apply automated fixes

* fix(job): utilize to_std err to check whether the time is past (#4370)

* [autofix.ci] apply automated fixes

* fix(test): use cloudflare workers-ai llms.txt for test

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
zwpaper added a commit that referenced this pull request Sep 25, 2025
* feat(index): indexing repo in shard when more than 20

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* chore: add env for enabling index repo in shard

# Conflicts:
#	ee/tabby-webserver/src/service/background_job/mod.rs

* [autofix.ci] apply automated fixes

* fix(job): utilize to_std err to check whether the time is past (#4370)

* [autofix.ci] apply automated fixes

* fix(test): use cloudflare workers-ai llms.txt for test

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants