Skip to content

Commit 47287c4

Browse files
authored
[BUG]: Fix compilation issues in statistics.rs (#5768)
## Description of changes Due to a merge conflict between PRs 5733 and 5735, `rust/worker/src/execution/functions/statistics.rs` is not compiling on main. This PR fixes that. - Improvements & Bug fixes - ... - New functionality - ... ## Test plan _How are these changes tested?_ - [ ] Tests pass locally with `pytest` for python, `yarn test` for js, `cargo test` for rust ## Migration plan _Are there any migrations, or any forwards/backwards compatibility changes needed in order to make sure this change deploys reliably?_ ## Observability plan _What is the plan to instrument and monitor this change?_ ## Documentation Changes _Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the_ [_docs section](https://github.com/chroma-core/chroma/tree/main/docs/docs.trychroma.com)?_
1 parent 81af3a0 commit 47287c4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

rust/worker/src/execution/functions/statistics.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ use chroma_types::{
1616
};
1717
use futures::StreamExt;
1818

19-
use crate::execution::operators::execute_task::TaskExecutor;
19+
use crate::execution::operators::execute_task::AttachedFunctionExecutor;
2020

2121
/// Create an accumulator for statistics.
2222
pub trait StatisticsFunctionFactory: std::fmt::Debug + Send + Sync {
@@ -170,7 +170,7 @@ impl Hash for StatisticsValue {
170170
pub struct StatisticsFunctionExecutor(pub Box<dyn StatisticsFunctionFactory>);
171171

172172
#[async_trait]
173-
impl TaskExecutor for StatisticsFunctionExecutor {
173+
impl AttachedFunctionExecutor for StatisticsFunctionExecutor {
174174
async fn execute(
175175
&self,
176176
input_records: Chunk<LogRecord>,

0 commit comments

Comments
 (0)