[GitHub] [spark] zhengruifeng commented on pull request #41379: [SPARK-43879][CONNECT] Decouple handle command and send response on server side

2023-06-05 Thread via GitHub
zhengruifeng commented on PR #41379: URL: https://github.com/apache/spark/pull/41379#issuecomment-1576583751 even if there is no other commands returning multiple response, I don't feel strongly about the idea of introducing such a limitation. I also think we should send them asap. --

[GitHub] [spark] zhengruifeng commented on pull request #41379: [SPARK-43879][CONNECT] Decouple handle command and send response on server side

2023-06-05 Thread via GitHub
zhengruifeng commented on PR #41379: URL: https://github.com/apache/spark/pull/41379#issuecomment-1576127284 cc @grundprinzip @hvanhovell what do you think of this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [spark] zhengruifeng commented on pull request #41379: [SPARK-43879][CONNECT] Decouple handle command and send response on server side

2023-05-30 Thread via GitHub
zhengruifeng commented on PR #41379: URL: https://github.com/apache/spark/pull/41379#issuecomment-1568420485 In case of multi `ExecutePlanResponse`s, does this change mean we can not send the first response before the last response to be ready? -- This is an automated message from the