Re: [PR] [SPARK-51804] Support `sample` in `DataFrame` [spark-connect-swift]

2025-04-15 Thread via GitHub
dongjoon-hyun closed pull request #59: [SPARK-51804] Support `sample` in `DataFrame` URL: https://github.com/apache/spark-connect-swift/pull/59 -- 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 sp

Re: [PR] [SPARK-51804] Support `sample` in `DataFrame` [spark-connect-swift]

2025-04-15 Thread via GitHub
dongjoon-hyun commented on PR #59: URL: https://github.com/apache/spark-connect-swift/pull/59#issuecomment-2804455498 Thank you! Merged to main. -- 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 t

Re: [PR] [SPARK-51804] Support `sample` in `DataFrame` [spark-connect-swift]

2025-04-15 Thread via GitHub
dongjoon-hyun commented on code in PR #59: URL: https://github.com/apache/spark-connect-swift/pull/59#discussion_r2044114774 ## Sources/SparkConnect/DataFrame.swift: ## @@ -297,6 +297,45 @@ public actor DataFrame: Sendable { return DataFrame(spark: self.spark, plan: Spark

Re: [PR] [SPARK-51804] Support `sample` in `DataFrame` [spark-connect-swift]

2025-04-15 Thread via GitHub
viirya commented on code in PR #59: URL: https://github.com/apache/spark-connect-swift/pull/59#discussion_r2043945625 ## Sources/SparkConnect/DataFrame.swift: ## @@ -297,6 +297,45 @@ public actor DataFrame: Sendable { return DataFrame(spark: self.spark, plan: SparkConnect

[PR] [SPARK-51804] Support `sample` in `DataFrame` [spark-connect-swift]

2025-04-15 Thread via GitHub
dongjoon-hyun opened a new pull request, #59: URL: https://github.com/apache/spark-connect-swift/pull/59 ### What changes were proposed in this pull request? ### Why are the changes needed? ### Does this PR introduce _any_ user-facing change?