Copilot commented on code in PR #8586:
URL: https://github.com/apache/texera/pull/8586#discussion_r4044226267
##########
agent-service/src/agent/texera-agent.ts:
##########
@@ -185,6 +186,7 @@ export class TexeraAgent {
userToken: this.delegateConfig.userToken,
workflowId: this.delegateConfig.workflowId,
computingUnitId: this.delegateConfig.computingUnitId,
+ warehouseId: this.delegateConfig.warehouseId,
Review Comment:
`warehouseId` can never reach this field from the new frontend payload. The
agent service's `POST /agents` schema,
`CreateAgentRequest`/`AgentDelegateConfig`, handler destructuring, and
`createAgentInstance` projection still only carry `computingUnitId`
(`agent-service/src/server.ts:85-91,178-201,231-236`;
`agent-service/src/types/agent.ts:118-124,147-153`). Consequently agent
executions still call the sync endpoint without a warehouse and are rejected
whenever warehouses are enabled. Thread `warehouseId` through that creation
path and add an end-to-end request-construction assertion.
--
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]
