Skip to content

Commit

Permalink
[BUG] Point query service to the right logservice port (#1921)
Browse files Browse the repository at this point in the history
## Description of changes

*Summarize the changes made by this PR.*
 - Improvements & Bug fixes
- Point the query service to port 50051 on the log service, not 50052.
We only use 50052 for the external-facing service during testing since
it conflicts with the sysdb on 50051.

## Test plan
*How are these changes tested?*

- [ ] Tests pass locally with `pytest` for python, `yarn test` for js,
`cargo test` for rust

## Documentation Changes
*Are all docstrings for user-facing APIs updated if required? Do we need
to make documentation changes in the [docs
repository](https://github.com/chroma-core/docs)?*
  • Loading branch information
beggers committed Mar 24, 2024
1 parent 8c7cc1b commit b602cfc
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion rust/worker/chroma_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ worker:
log:
Grpc:
host: "logservice.chroma"
port: 50052
port: 50051
dispatcher:
num_worker_threads: 4
dispatcher_queue_size: 100
Expand Down
8 changes: 4 additions & 4 deletions rust/worker/src/config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ mod tests {
log:
Grpc:
host: "localhost"
port: 50052
port: 50051
dispatcher:
num_worker_threads: 4
dispatcher_queue_size: 100
Expand Down Expand Up @@ -217,7 +217,7 @@ mod tests {
log:
Grpc:
host: "localhost"
port: 50052
port: 50051
dispatcher:
num_worker_threads: 4
dispatcher_queue_size: 100
Expand Down Expand Up @@ -285,7 +285,7 @@ mod tests {
log:
Grpc:
host: "localhost"
port: 50052
port: 50051
dispatcher:
num_worker_threads: 4
dispatcher_queue_size: 100
Expand Down Expand Up @@ -333,7 +333,7 @@ mod tests {
log:
Grpc:
host: "localhost"
port: 50052
port: 50051
dispatcher:
num_worker_threads: 4
dispatcher_queue_size: 100
Expand Down

0 comments on commit b602cfc

Please sign in to comment.