This is an automated email from the ASF dual-hosted git repository.
github-merge-queue[bot] pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/texera.git
The following commit(s) were added to refs/heads/main by this push:
new e03d97108d refactor(agent-service): remove unused targetOperatorId
from toLogicalPlan (#7430)
e03d97108d is described below
commit e03d97108ddd47befc78a87f467f781dd50a56e6
Author: Ghulam Mustafa <[email protected]>
AuthorDate: Sun Aug 9 16:26:22 2026 +0500
refactor(agent-service): remove unused targetOperatorId from toLogicalPlan
(#7430)
### What changes were proposed in this PR?
It Refactors `WorkflowState.toLogicalPlan()` by removing the unused
`targetOperatorId` parameter.
The parameter was never referenced in the method (it always built the
whole-graph plan) and the only caller (`texera-agent.ts`) passed no
arguments to it. The need to fetch a sub-graph of a target is already
properly served by `getSubDAG(targetOperatorId)`.
### Any related issues, documentation, discussions?
Closes #7170
### How was this PR tested?
Run the TypeScript compiler checks and existing agent-service unit tests
to verify no callers or types were broken.
### Was this PR authored or co-authored using generative AI tooling?
Generated-by: Gemini 3.1 Pro (High)
---
agent-service/src/agent/workflow-state.ts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/agent-service/src/agent/workflow-state.ts
b/agent-service/src/agent/workflow-state.ts
index 04ad2b0e4e..d53692f33c 100644
--- a/agent-service/src/agent/workflow-state.ts
+++ b/agent-service/src/agent/workflow-state.ts
@@ -415,7 +415,7 @@ export class WorkflowState {
this.settings = content.settings ? { ...content.settings } : {
...DEFAULT_WORKFLOW_SETTINGS };
}
- toLogicalPlan(targetOperatorId?: string): LogicalPlan {
+ toLogicalPlan(): LogicalPlan {
const enabledOperators = this.getAllEnabledOperators();
const operators: LogicalOperator[] = enabledOperators.map(op => ({
