Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18942
Thank you, @poplav ! :D
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
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 user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18942
Ping, @poplav .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18942
Gentle ping, @poplav .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18942
Hi, @poplav . Unfortunately, this seems to be too old to be merged. Could
you close this PR?
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18942
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18942
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18942
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18942
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18942
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
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, but
Github user pmishr1 commented on the issue:
https://github.com/apache/spark/pull/18942
@poplav were you able to patch this PR and build successfully on top of
2.1.1 ?
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18942
You can patch it to your forked version.
---
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 project does not have this
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 user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18942
@poplav This is not a regression from 2.0, right?
Since we might not release 2.1.2, this PR might not be merged to upstream
after a discussion with @zsxwing Maybe you can patch it in
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18942
@poplav it looks good
@gatorsmile Do you think it is ok for backport now? The previous commit
included unnecessary changes.
---
If your project is set up for it, you can reply to this email and
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 user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18942
This might be too risky to be merged to 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 project does not have this
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 user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18942
Can one of the admins verify this patch?
---
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 project does not have this
20 matches
Mail list logo