abhinavgautam01 commented on code in PR #1685:
URL:
https://github.com/apache/datafusion-ballista/pull/1685#discussion_r3223138965
##########
ballista/executor/src/executor_server.rs:
##########
@@ -148,6 +169,8 @@ pub async fn startup<T: 'static + AsLogicalPlan, U: 'static
+ AsExecutionPlan>(
})
};
+ wait_executor_grpc_listen(&config.bind_host, config.grpc_port).await?;
Review Comment:
For the real executor process they stay aligned because
structure_executor_metadata copies options.grpc_port into protobuf grpc_port;
startup still has a debug_assert_eq! linking executor.metadata.grpc_port and
config.grpc_port. Standalone push builds ExecutorRegistration and
ExecutorProcessConfig from the same bound TcpListener port.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
