Skip to content

Commit

Permalink
Fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
willdealtry committed Jan 6, 2025
1 parent 66f6264 commit 97629cb
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion cpp/arcticdb/version/version_core.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -558,7 +558,7 @@ folly::Future<std::vector<EntityId>> schedule_clause_processing(
auto entity_ids_vec_fut = folly::collect(*futures).via(&async::io_executor());

const auto scheduling_iterations = num_scheduling_iterations(*clauses);
for (auto i = 0UL; i < scheduling_iterations; ++i) {
for (auto i = 1UL; i < scheduling_iterations; ++i) {
entity_ids_vec_fut = std::move(entity_ids_vec_fut).thenValue([clauses, scheduling_iterations, i] (std::vector<std::vector<EntityId>>&& entity_id_vectors) {
ARCTICDB_RUNTIME_DEBUG(log::memory(), "Scheduling iteration {} of {}", i, scheduling_iterations);

Expand Down
2 changes: 0 additions & 2 deletions python/arcticdb/version_store/processing.py
Original file line number Diff line number Diff line change
Expand Up @@ -585,7 +585,6 @@ def agg(self, aggregations: Dict[str, Union[str, Tuple[str, str]]]):
self._python_clauses[-1].aggregations = aggregations
return self


def resample(
self,
rule: Union[str, pd.DateOffset],
Expand Down Expand Up @@ -780,7 +779,6 @@ def resample(
self._python_clauses = self._python_clauses + [PythonResampleClause(rule=rule, closed=boundary_map[closed], label=boundary_map[label], offset=offset_ns, origin=origin)]
return self


# TODO: specify type of other must be QueryBuilder with from __future__ import annotations once only Python 3.7+
# supported
def then(self, other):
Expand Down

0 comments on commit 97629cb

Please sign in to comment.