Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20984
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r181339660
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,46 @@ abstract class ArrayData extends Specializ
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r181140913
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/ArrayDataIndexedSeqSuite.scala
---
@@ -0,0 +1,69 @@
+/*
+ * Licensed
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r181140698
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,46 @@ abstract class ArrayData extends Sp
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r180420612
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,46 @@ abstract class ArrayData extends Specializ
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r180364984
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,46 @@ abstract class ArrayData extends Speci
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r180361913
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,40 @@ abstract class ArrayData extends Specializ
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r180360146
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,40 @@ abstract class ArrayData extends Speci
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r180358272
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/ArrayDataIndexedSeqSuite.scala
---
@@ -0,0 +1,69 @@
+/*
+ * Licensed to the
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r180357829
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,37 @@ abstract class ArrayData extends Specializ
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r180292435
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,32 @@ abstract class ArrayData extends Specializ
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20984#discussion_r179920210
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayData.scala
---
@@ -164,3 +167,32 @@ abstract class ArrayData extends Specialize
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/20984
[SPARK-23875][SQL] Add IndexedSeq wrapper for ArrayData
## What changes were proposed in this pull request?
We don't have a good way to sequentially access `UnsafeArrayData` with a
common i
13 matches
Mail list logo