Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84588059
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystTypeConverters.scala
---
@@ -433,18 +413,12 @@ object
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84587828
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystTypeConverters.scala
---
@@ -433,18 +413,12 @@ object CatalystTypeConverters
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84569857
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapData.scala
---
@@ -35,6 +39,23 @@ object ArrayBasedMapData {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84394697
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapData.scala
---
@@ -35,6 +39,23 @@ object ArrayBasedMapData {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84394627
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapData.scala
---
@@ -35,6 +39,23 @@ object ArrayBasedMapData {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84394511
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapData.scala
---
@@ -35,6 +39,23 @@ object ArrayBasedMapData {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84395758
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapData.scala
---
@@ -35,6 +39,23 @@ object ArrayBasedMapData {
GitHub user tejasapatil opened a pull request:
https://github.com/apache/spark/pull/15573
[SPARK-18035] [SQL] Unwrapping java maps in HiveInspectors allocates
unnecessary buffer
## What changes were proposed in this pull request?
Jira: