dongjoon-hyun commented on PR #68: URL: https://github.com/apache/spark-connect-swift/pull/68#issuecomment-2814663320
Thank you! You are right. I addressed your comment by fixing it and adding a test coverage, @viirya . > Do we need to set isAll: true for unionByName like union? -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org