Skip to content

Commit

Permalink
fix invariant in batch tick loader (#7280)
Browse files Browse the repository at this point in the history
  • Loading branch information
prha committed Apr 5, 2022
1 parent ce69cc1 commit 052bf22
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -219,23 +219,17 @@ def get_sensor_state(self, sensor_state):
states = self._get(RepositoryDataType.SENSOR_STATES, sensor_state, 1)
return states[0] if states else None

def get_sensor_ticks(self, origin_id, limit):
def get_sensor_ticks(self, origin_id, selector_id, limit):
check.invariant(
origin_id
in [
sensor.get_external_origin_id()
for sensor in self._repository.get_external_sensors()
]
selector_id
in [sensor.selector_id for sensor in self._repository.get_external_sensors()]
)
return self._get(RepositoryDataType.SENSOR_TICKS, origin_id, limit)

def get_schedule_ticks(self, origin_id, limit):
def get_schedule_ticks(self, origin_id, selector_id, limit):
check.invariant(
origin_id
in [
schedule.get_external_origin_id()
for schedule in self._repository.get_external_schedules()
]
selector_id
in [schedule.selector_id for schedule in self._repository.get_external_schedules()]
)
return self._get(RepositoryDataType.SCHEDULE_TICKS, origin_id, limit)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -394,11 +394,15 @@ def resolve_ticks(
if self._batch_loader and limit and not cursor and not before and not after:
ticks = (
self._batch_loader.get_sensor_ticks(
self._instigator_state.instigator_origin_id, limit
self._instigator_state.instigator_origin_id,
self._instigator_state.selector_id,
limit,
)
if self._instigator_state.instigator_type == InstigatorType.SENSOR
else self._batch_loader.get_schedule_ticks(
self._instigator_state.instigator_origin_id, limit
self._instigator_state.instigator_origin_id,
self._instigator_state.selector_id,
limit,
)
)
return [GrapheneInstigationTick(graphene_info, tick) for tick in ticks]
Expand Down

0 comments on commit 052bf22

Please sign in to comment.