[GitHub] spark issue #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a Java exc...

2018-09-27 Thread poplav
Github user poplav commented on the issue: https://github.com/apache/spark/pull/18942 Was away on vacation. Closing PR, thanks. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark pull request #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a J...

2018-09-27 Thread poplav
Github user poplav closed the pull request at: https://github.com/apache/spark/pull/18942 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark issue #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a Java exc...

2017-10-12 Thread poplav
Github user poplav commented on the issue: https://github.com/apache/spark/pull/18942 @pmishr1 Yeah it worked this PR/branch needs one more commit that is at https://github.com/poplav/spark/commit/48ea442ea670cc612b91ad037b223750aa70b918. I would update this PR with that commit

[GitHub] spark issue #16087: [SPARK-18656][SQL] Memory efficient implementation of th...

2017-08-23 Thread poplav
Github user poplav commented on the issue: https://github.com/apache/spark/pull/16087 @barrybecker4 Following the trail of this issue...What is the solution for many columns rather than the workaround for chunking? Thanks! --- If your project is set up for it, you can reply

[GitHub] spark issue #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a Java exc...

2017-08-15 Thread poplav
Github user poplav commented on the issue: https://github.com/apache/spark/pull/18942 Was this working in 2.0 in the first place? I want to get this into 2.1.1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark issue #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a Java exc...

2017-08-15 Thread poplav
Github user poplav commented on the issue: https://github.com/apache/spark/pull/18942 @kiszk , I updated the PR to remove the `prunePartionsByFilter` bit. Please let me know now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a J...

2017-08-15 Thread poplav
Github user poplav commented on a diff in the pull request: https://github.com/apache/spark/pull/18942#discussion_r133180913 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogUtils.scala --- @@ -123,6 +124,38 @@ object ExternalCatalogUtils

[GitHub] spark issue #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a Java exc...

2017-08-14 Thread poplav
Github user poplav commented on the issue: https://github.com/apache/spark/pull/18942 Looking back at this. I simply cherry picked the commit from the branch, there appears to be more to this backport. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark issue #17087: [SPARK-19372][SQL] Fix throwing a Java exception at df.f...

2017-08-14 Thread poplav
Github user poplav commented on the issue: https://github.com/apache/spark/pull/17087 Hi, All. I am trying to get this included into Spark 2.1.1. I opened a PR https://github.com/apache/spark/pull/18942. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request #18942: [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a J...

2017-08-14 Thread poplav
GitHub user poplav opened a pull request: https://github.com/apache/spark/pull/18942 [BACKPORT-2.1][SPARK-19372][SQL] Fix throwing a Java exception at df.fliter() due to 64KB bytecode size limit ### What changes were proposed in this pull request? This PR is backport