weiqingy commented on code in PR #882:
URL: https://github.com/apache/flink-agents/pull/882#discussion_r3547749652


##########
docs/content/docs/operations/configuration.md:
##########


Review Comment:
   @wenjin272  Thanks for the review. Both addressed:
   
   - **`configuration.md` Local-mode example**: removed in `b495072`. It 
pointed at the run-without-Flink path this PR deletes. That left MiniCluster as 
the only special case, so I also changed the intro from "the following two 
cases" to the singular form.
   - **`LocalMemoryObject`**: moved to 
`flink_agents.runtime.tests.local_memory_object` in `d4243f1`, so it's no 
longer in the production runtime package. It's now purely an in-memory test 
fixture, and the three importers (`test_chat_model_action`, 
`test_memory_reference`, `test_local_memory_object`) are updated — it now sits 
alongside its own unit test in `runtime/tests`.
   



-- 
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]

Reply via email to