Skip to content

Commit d70110e

Browse files
committed
fixup! [runtime][java] Support use Long-Term Memory in action.
address comments.
1 parent 27776b1 commit d70110e

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

e2e-test/flink-agents-end-to-end-tests-integration/src/test/java/org/apache/flink/agents/integration/test/VectorStoreLongTermMemoryTest.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,8 @@
5050
import org.junit.jupiter.api.Test;
5151
import org.junit.jupiter.api.TestInfo;
5252
import org.mockito.Mockito;
53+
import org.slf4j.Logger;
54+
import org.slf4j.LoggerFactory;
5355

5456
import java.util.ArrayList;
5557
import java.util.Collections;
@@ -71,6 +73,8 @@
7173
*/
7274
@Disabled("Should setup Elasticsearch server.")
7375
public class VectorStoreLongTermMemoryTest {
76+
private static final Logger LOG = LoggerFactory.getLogger(VectorStoreLongTermMemoryTest.class);
77+
7478
private static final String NAME = "chat-history";
7579
private final VectorStoreLongTermMemory ltm;
7680
private MemorySet memorySet;
@@ -318,7 +322,7 @@ public void testUsingLtmInAction() throws Exception {
318322
agentsEnv.execute();
319323

320324
// check async compaction
321-
System.out.print(es.get(null, "ltm_test_job-2-test-ltm", Collections.emptyMap()));
325+
LOG.debug(es.get(null, "ltm_test_job-2-test-ltm", Collections.emptyMap()).toString());
322326
Assertions.assertEquals(1, es.size("ltm_test_job-2-test-ltm"));
323327
} finally {
324328
es.deleteCollection("ltm_test_job-1-test-ltm");

runtime/src/main/java/org/apache/flink/agents/runtime/context/RunnerContextImpl.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
import org.apache.flink.agents.plan.AgentPlan;
3131
import org.apache.flink.agents.plan.utils.JsonUtils;
3232
import org.apache.flink.agents.runtime.memory.CachedMemoryStore;
33+
import org.apache.flink.agents.runtime.memory.KeyedMemory;
3334
import org.apache.flink.agents.runtime.memory.MemoryObjectImpl;
3435
import org.apache.flink.agents.runtime.memory.VectorStoreLongTermMemory;
3536
import org.apache.flink.agents.runtime.metrics.FlinkAgentsMetricGroupImpl;
@@ -107,7 +108,7 @@ public void switchActionContext(String actionName, MemoryContext memoryContext,
107108
this.actionName = actionName;
108109
this.memoryContext = memoryContext;
109110
if (ltm != null) {
110-
ltm.switchContext(key);
111+
((KeyedMemory) ltm).switchContext(key);
111112
}
112113
}
113114

0 commit comments

Comments
 (0)