- Notifications
You must be signed in to change notification settings - Fork929
fix(coderd/database): consider tag sets when calculating queue position#16685
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes from1 commit
Commits
Show all changes
20 commits Select commitHold shift + click to select a range
1e35c53
fix(coderd/database): remove linux build tags from db package
evgeniy-scherbina5c49221
fix(coderd/database): consider tag sets when calculating queue position
evgeniy-scherbina4257ca0
test(coderd/database): skip tests when PostgreSQL is unavailable
evgeniy-scherbina30f007c
test(coderd/database): create default provisioner in tests
evgeniy-scherbinaacb93ac
fix(coderd/database): correctly calculate pending and ranked jobs
evgeniy-scherbina5aa0ffa
test(coderd/database): improve test coverage
evgeniy-scherbinac1f421d
refactor(coderd/database): use empty tag sets
evgeniy-scherbinae7693ee
test(coderd/database): improve test coverage
evgeniy-scherbinaa9ed7d2
test(coderd/database): wrap test cases in subtests
evgeniy-scherbina574cdf6
refactor(coderd/database): minor refactoring in tests
evgeniy-scherbina8b3446e
fix(coderd/database): linter
evgeniy-scherbinac4316fd
refactor(coderd/database): use job_status generated column
evgeniy-scherbinab4ea4db
perf: add index on provisioner_jobs table for status field
evgeniy-scherbina675b3e9
Merge remote-tracking branch 'origin/main' into 15843-queue-position
evgeniy-scherbina1f8e4e3
fix(coderd/database): increment migration number
evgeniy-scherbinaf9c9711
fix(coderd/database): update dump.sql
evgeniy-scherbina61a9f58
perf: optimize get-queue-position sql query
evgeniy-scherbina4f77f67
chore: update dbmem
evgeniy-scherbina359a9e0
Merge remote-tracking branch 'origin/main' into 15843-queue-position
evgeniy-scherbina6f4da84
fix(coderd/database): increment migration number
evgeniy-scherbinaFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
NextNext commit
fix(coderd/database): remove linux build tags from db package
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
commit1e35c53c692ffd9cc733de65aa0fd23250a1b401
There are no files selected for viewing
2 changes: 0 additions & 2 deletionscoderd/database/db_test.go
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
package database_test | ||
import ( | ||
2 changes: 0 additions & 2 deletionscoderd/database/dbtestutil/postgres_test.go
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
package dbtestutil_test | ||
import ( | ||
2 changes: 0 additions & 2 deletionscoderd/database/migrations/migrate_test.go
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
package migrations_test | ||
import ( | ||
2 changes: 0 additions & 2 deletionscoderd/database/pubsub/pubsub_linux_test.go
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
package pubsub_test | ||
import ( | ||
2 changes: 0 additions & 2 deletionscoderd/database/querier_test.go
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
package database_test | ||
import ( | ||
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.