Ryan-Nightwish commented on PR #830:
URL: https://github.com/apache/flink-agents/pull/830#issuecomment-4707729373
Hi @xintongsong , I made the following additional changes:
- **Simplified sensory memory access**: Replaced the single "res" key
(which packed all state into one JSON-serialized blob in Python, or
one nested Map in Java) with path-based individual keys supported by
the MemoryObject API. Each field now has its own key: primitives "id"
and "text" are stored directly. This removes all json.dumps /
json.loads calls from the Python side and eliminates the unchecked Map
casts from the Java side.
- **Removed now-redundant helpers**: _init_row / initRow,
_all_aspects_received / allAspectsReceived are inlined or dropped;
_build_summarize_request and _build_output_event now take typed
arguments directly instead of receiving the whole row dict / Map.
- **Updated docs**: The parallel_llm.md quickstart code blocks and
Constraints hint are updated to reflect the new memory access pattern.
Please take a look and let me know if there is anything else to adjust.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
