dongjoon-hyun closed pull request #6: [SPARK-51465] Use `Apache Arrow Swift`
19.0.1
URL: https://github.com/apache/spark-connect-swift/pull/6
--
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 spec
dongjoon-hyun commented on PR #6:
URL:
https://github.com/apache/spark-connect-swift/pull/6#issuecomment-2715038929
Thank you so much all for being interested in this new codebase and taking a
look at this to help, @MaxGekk , @viirya and @huaxingao .
Merged to main.
--
This is an
dongjoon-hyun commented on PR #6:
URL:
https://github.com/apache/spark-connect-swift/pull/6#issuecomment-2715003949
After building the initial working code, we are going to entering QA period
by adding more unit test coverages. And, all issues are going to the upstreams
because we don't wa
dongjoon-hyun commented on PR #6:
URL:
https://github.com/apache/spark-connect-swift/pull/6#issuecomment-271568
Just to give the reviewers the background, `Swift 6` compiler becomes more
like `Rust` compiler in terms of `Concurrency and Data Safety` check.
- https://www.swift.org/blo
dongjoon-hyun commented on PR #6:
URL:
https://github.com/apache/spark-connect-swift/pull/6#issuecomment-2714994992
Thank you for review, @viirya . Yes, it's required when Apache Arrow starts
to support Swift 6.0.
--
This is an automated message from the Apache Git Service.
To respond to
viirya commented on PR #6:
URL:
https://github.com/apache/spark-connect-swift/pull/6#issuecomment-2714985680
For the changes, do you plan to propose them to Arrow Swift repo?
```swift
- public enum ArrowTypeId {
+ public enum ArrowTypeId: Sendable {
- public enum Info {
MaxGekk commented on code in PR #6:
URL: https://github.com/apache/spark-connect-swift/pull/6#discussion_r1989490231
##
Sources/SparkConnect/ArrowArray.swift:
##
@@ -0,0 +1,331 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agre
dongjoon-hyun commented on code in PR #6:
URL: https://github.com/apache/spark-connect-swift/pull/6#discussion_r1989531383
##
Sources/SparkConnect/ArrowArray.swift:
##
@@ -0,0 +1,331 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
dongjoon-hyun commented on code in PR #6:
URL: https://github.com/apache/spark-connect-swift/pull/6#discussion_r1989528793
##
Sources/SparkConnect/ArrowArray.swift:
##
@@ -0,0 +1,331 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
dongjoon-hyun commented on PR #6:
URL:
https://github.com/apache/spark-connect-swift/pull/6#issuecomment-2714672334
Could you review this PR when you have some time, @huaxingao ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
dongjoon-hyun opened a new pull request, #6:
URL: https://github.com/apache/spark-connect-swift/pull/6
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
dongjoon-hyun commented on PR #6:
URL:
https://github.com/apache/spark-connect-swift/pull/6#issuecomment-2712886857
Could you review this PR to use `Apache Arrow Swift` source code in `Apache
Spark Connect for Swift` when you have some time, @MaxGekk ? This is a temporal
usage until Apache
12 matches
Mail list logo
