Skip to content

Commit 8f7c562

Browse files
author
Rares Polenciuc
committed
feat: execution state pagination and token validation
- Add paging logic in checkpoint processor with next_marker support - Implement checkpoint token validation - Add token expiration checking with error responses - Handle missing token cases with context-appropriate validation - Add pagination metadata to responses with configurable max_items - Add test coverage for all validation scenarios
1 parent fdbb2ac commit 8f7c562

File tree

8 files changed

+364
-61
lines changed

8 files changed

+364
-61
lines changed

src/aws_durable_execution_sdk_python_testing/checkpoint/processor.py

Lines changed: 29 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
CheckpointUpdatedExecutionState,
1010
OperationUpdate,
1111
StateOutput,
12+
Operation,
1213
)
1314

1415
from aws_durable_execution_sdk_python_testing.checkpoint.transformer import (
@@ -88,14 +89,37 @@ def process_checkpoint(
8889
def get_execution_state(
8990
self,
9091
checkpoint_token: str,
91-
next_marker: str, # noqa: ARG002
92-
max_items: int = 1000, # noqa: ARG002
92+
next_marker: str | None = None,
93+
max_items: int = 1000,
9394
) -> StateOutput:
94-
"""Get current execution state."""
95+
"""Get current execution state with batched checkpoint token validation and pagination."""
96+
if not checkpoint_token:
97+
msg: str = "Checkpoint token is required"
98+
raise InvalidParameterValueException(msg)
99+
95100
token: CheckpointToken = CheckpointToken.from_str(checkpoint_token)
96101
execution: Execution = self._store.load(token.execution_arn)
102+
execution.validate_checkpoint_token(checkpoint_token)
103+
104+
# Get all operations
105+
all_operations: list[Operation] = execution.get_navigable_operations()
106+
107+
# Apply pagination
108+
start_index: int = 0
109+
if next_marker:
110+
try:
111+
start_index = int(next_marker)
112+
except ValueError:
113+
start_index = 0
114+
115+
end_index: int = start_index + max_items
116+
paginated_operations: list[Operation] = all_operations[start_index:end_index]
117+
118+
# Determine next marker
119+
next_marker_result: str | None = (
120+
str(end_index) if end_index < len(all_operations) else None
121+
)
97122

98-
# TODO: paging when size or max
99123
return StateOutput(
100-
operations=execution.get_navigable_operations(), next_marker=None
124+
operations=paginated_operations, next_marker=next_marker_result
101125
)

src/aws_durable_execution_sdk_python_testing/checkpoint/processors/execution.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,8 @@ def process(
3838
)
3939
case _:
4040
# intentional. actual service will fail any EXECUTION update that is not SUCCEED.
41-
error = (
42-
update.error
43-
if update.error
44-
else ErrorObject.from_message(
45-
"There is no error details but EXECUTION checkpoint action is not SUCCEED."
46-
)
41+
error = update.error or ErrorObject.from_message(
42+
"There is no error details but EXECUTION checkpoint action is not SUCCEED."
4743
)
4844
# All EXECUTION failures go through normal fail path
4945
# Timeout/Stop status is set by executor based on the operation that caused it

src/aws_durable_execution_sdk_python_testing/checkpoint/transformer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def __init__(
5555
self,
5656
processors: MutableMapping[OperationType, OperationProcessor] | None = None,
5757
):
58-
self.processors = processors if processors else self._DEFAULT_PROCESSORS
58+
self.processors = processors or self._DEFAULT_PROCESSORS
5959

6060
def process_updates(
6161
self,

src/aws_durable_execution_sdk_python_testing/execution.py

Lines changed: 32 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ def __init__(
6666
self.start_input: StartDurableExecutionInput = start_input
6767
self.operations: list[Operation] = operations
6868
self.updates: list[OperationUpdate] = []
69-
self.used_tokens: set[str] = set()
69+
self.generated_tokens: set[str] = set()
7070
# TODO: this will need to persist/rehydrate depending on inmemory vs sqllite store
7171
self._token_sequence: int = 0
7272
self._state_lock: Lock = Lock()
@@ -122,7 +122,7 @@ def to_dict(self) -> dict[str, Any]:
122122
"StartInput": self.start_input.to_dict(),
123123
"Operations": [op.to_dict() for op in self.operations],
124124
"Updates": [update.to_dict() for update in self.updates],
125-
"UsedTokens": list(self.used_tokens),
125+
"GeneratedTokens": list(self.generated_tokens),
126126
"TokenSequence": self._token_sequence,
127127
"IsComplete": self.is_complete,
128128
"Result": self.result.to_dict() if self.result else None,
@@ -150,7 +150,7 @@ def from_dict(cls, data: dict[str, Any]) -> Execution:
150150
execution.updates = [
151151
OperationUpdate.from_dict(update_data) for update_data in data["Updates"]
152152
]
153-
execution.used_tokens = set(data["UsedTokens"])
153+
execution.generated_tokens = set(data["GeneratedTokens"])
154154
execution._token_sequence = data["TokenSequence"] # noqa: SLF001
155155
execution.is_complete = data["IsComplete"]
156156
execution.result = (
@@ -205,13 +205,41 @@ def get_new_checkpoint_token(self) -> str:
205205
token_sequence=new_token_sequence,
206206
)
207207
token_str = token.to_str()
208-
self.used_tokens.add(token_str)
208+
self.generated_tokens.add(token_str)
209209
return token_str
210210

211211
def get_navigable_operations(self) -> list[Operation]:
212212
"""Get list of operations, but exclude child operations where the parent has already completed."""
213213
return self.operations
214214

215+
def validate_checkpoint_token(
216+
self,
217+
token: str | None,
218+
is_required: bool = True, # noqa: FBT001, FBT002
219+
checkpoint_required_msg: str | None = None,
220+
) -> None:
221+
"""Validate checkpoint token against this execution."""
222+
if not token:
223+
if is_required:
224+
msg: str = checkpoint_required_msg or "Checkpoint token is required"
225+
raise InvalidParameterValueException(msg)
226+
return
227+
228+
checkpoint_token: CheckpointToken = CheckpointToken.from_str(token)
229+
if checkpoint_token.execution_arn != self.durable_execution_arn:
230+
msg = "Checkpoint token does not match execution ARN"
231+
raise InvalidParameterValueException(msg)
232+
233+
if self.is_complete or checkpoint_token.token_sequence > self.token_sequence:
234+
msg = "Invalid or expired checkpoint token"
235+
raise InvalidParameterValueException(msg)
236+
237+
# Check if token has been generated
238+
token_str: str = checkpoint_token.to_str()
239+
if token_str not in self.generated_tokens:
240+
msg = f"Invalid checkpoint token: {token_str}"
241+
raise InvalidParameterValueException(msg)
242+
215243
def get_assertable_operations(self) -> list[Operation]:
216244
"""Get list of operations, but exclude the EXECUTION operations"""
217245
# TODO: this excludes EXECUTION at start, but can there be an EXECUTION at the end if there was a checkpoint with large payload?

src/aws_durable_execution_sdk_python_testing/executor.py

Lines changed: 20 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,8 @@
5151
if TYPE_CHECKING:
5252
from collections.abc import Awaitable, Callable
5353

54+
from aws_durable_execution_sdk_python.lambda_service import Operation
55+
5456
from aws_durable_execution_sdk_python_testing.invoker import Invoker
5557
from aws_durable_execution_sdk_python_testing.scheduler import Event, Scheduler
5658
from aws_durable_execution_sdk_python_testing.stores.base import ExecutionStore
@@ -311,32 +313,33 @@ def get_execution_state(
311313
ResourceNotFoundException: If execution does not exist
312314
InvalidParameterValueException: If checkpoint token is invalid
313315
"""
314-
execution = self.get_execution(execution_arn)
315-
316-
# TODO: Validate checkpoint token if provided
317-
if checkpoint_token and checkpoint_token not in execution.used_tokens:
318-
msg: str = f"Invalid checkpoint token: {checkpoint_token}"
319-
raise InvalidParameterValueException(msg)
316+
execution: Execution = self.get_execution(execution_arn)
317+
checkpoint_required_msg: str = (
318+
"Checkpoint token is required for paginated requests on active executions"
319+
)
320+
checkpoint_required: bool = not execution.is_complete and marker is not None
321+
execution.validate_checkpoint_token(
322+
checkpoint_token, checkpoint_required, checkpoint_required_msg
323+
)
320324

321325
# Get operations (excluding the initial EXECUTION operation for state)
322-
operations = execution.get_assertable_operations()
326+
operations: list[Operation] = execution.get_assertable_operations()
323327

324328
# Apply pagination
325329
if max_items is None:
326330
max_items = 100
327331

328-
# Simple pagination - in real implementation would need proper marker handling
329-
start_index = 0
332+
start_index: int = 0
330333
if marker:
331334
try:
332335
start_index = int(marker)
333336
except ValueError:
334337
start_index = 0
335338

336-
end_index = start_index + max_items
337-
paginated_operations = operations[start_index:end_index]
339+
end_index: int = start_index + max_items
340+
paginated_operations: list[Operation] = operations[start_index:end_index]
338341

339-
next_marker = None
342+
next_marker: str | None = None
340343
if end_index < len(operations):
341344
next_marker = str(end_index)
342345

@@ -468,11 +471,11 @@ def checkpoint_execution(
468471
InvalidParameterValueException: If checkpoint token is invalid
469472
"""
470473
execution = self.get_execution(execution_arn)
471-
472-
# Validate checkpoint token
473-
if checkpoint_token not in execution.used_tokens:
474-
msg: str = f"Invalid checkpoint token: {checkpoint_token}"
475-
raise InvalidParameterValueException(msg)
474+
execution.validate_checkpoint_token(
475+
checkpoint_token,
476+
is_required=True,
477+
checkpoint_required_msg="Checkpoint token is required for checkpoint operations",
478+
)
476479

477480
# TODO: Process operation updates using the checkpoint processor
478481
# This would integrate with the existing checkpoint processing pipeline

tests/execution_test.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def test_execution_init():
3939
assert execution.start_input == start_input
4040
assert execution.operations == operations
4141
assert execution.updates == []
42-
assert execution.used_tokens == set()
42+
assert execution.generated_tokens == set()
4343
assert execution.token_sequence == 0
4444
assert execution.is_complete is False
4545
assert execution.consecutive_failed_invocation_attempts == 0
@@ -150,8 +150,8 @@ def test_get_new_checkpoint_token():
150150
token2 = execution.get_new_checkpoint_token()
151151

152152
assert execution.token_sequence == 2
153-
assert token1 in execution.used_tokens
154-
assert token2 in execution.used_tokens
153+
assert token1 in execution.generated_tokens
154+
assert token2 in execution.generated_tokens
155155
assert token1 != token2
156156

157157

0 commit comments

Comments
 (0)