[GitHub] spark pull request #19810: [SPARK-22599][SQL] In-Memory Table Pruning withou...

2017-12-04 Thread sadikovi
Github user sadikovi commented on a diff in the pull request: https://github.com/apache/spark/pull/19810#discussion_r154848470 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala --- @@ -193,38 +195,68 @@ case class

[GitHub] spark pull request #19810: [SPARK-22599][SQL] In-Memory Table Pruning withou...

2017-12-04 Thread sadikovi
Github user sadikovi commented on a diff in the pull request: https://github.com/apache/spark/pull/19810#discussion_r154815012 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala --- @@ -193,38 +195,68 @@ case class

[GitHub] spark pull request #19810: [SPARK-22599][SQL] In-Memory Table Pruning withou...

2017-12-04 Thread sadikovi
Github user sadikovi commented on a diff in the pull request: https://github.com/apache/spark/pull/19810#discussion_r154814573 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala --- @@ -52,6 +52,68 @@ object InMemoryRelation

[GitHub] spark issue #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus event que...

2017-07-04 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/18476 @tgravescs @JoshRosen could you review this PR again, please? Thanks. --- 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 #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus event que...

2017-06-30 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/18476 @JoshRosen Thank you for the comment! I updated config option name to reflect changes in master branch. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus event que...

2017-06-30 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/18476 @tgravescs I updated description with latest suggestions. Would appreciate, if you could review again. Thanks! --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus ev...

2017-06-30 Thread sadikovi
Github user sadikovi commented on a diff in the pull request: https://github.com/apache/spark/pull/18476#discussion_r125138807 --- Diff: docs/configuration.md --- @@ -1398,6 +1398,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus ev...

2017-06-30 Thread sadikovi
Github user sadikovi commented on a diff in the pull request: https://github.com/apache/spark/pull/18476#discussion_r125137670 --- Diff: docs/configuration.md --- @@ -1398,6 +1398,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus ev...

2017-06-30 Thread sadikovi
Github user sadikovi commented on a diff in the pull request: https://github.com/apache/spark/pull/18476#discussion_r125133239 --- Diff: docs/configuration.md --- @@ -1398,6 +1398,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark issue #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus event que...

2017-06-29 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/18476 @srowen Could you review this pull request, please? Thank you. --- 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

[GitHub] spark pull request #18476: [SPARK-20858][DOC][MINOR] Document ListenerBus ev...

2017-06-29 Thread sadikovi
GitHub user sadikovi opened a pull request: https://github.com/apache/spark/pull/18476 [SPARK-20858][DOC][MINOR] Document ListenerBus event queue size ## What changes were proposed in this pull request? This change adds a new configuration option

[GitHub] spark issue #15134: [SPARK-17580][CORE]Add random UUID as app name while app...

2016-09-18 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/15134 @phalodi Does this solve (intend to solve) situation when spark-submit is launched with empty app name? Currently, as of 1.6, it will use empty application name. --- If your project is set up

[GitHub] spark issue #13614: Support Stata-like tabulation of values in a single colu...

2016-06-12 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/13614 @shafiquejamal The PR title should be of the form [SPARK-] [COMPONENT] Title, where SPARK- is the relevant JIRA number, COMPONENT is one of the PR categories shown at https://spark

[GitHub] spark issue #13563: [SPARK-15826] [CORE] PipedRDD to strictly use UTF-8 and ...

2016-06-10 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/13563 Can you give an example where it would not work? I was actually thinking about adding codec for stderr with malformed rule of CodingErrorAction.REPLACE to make it safer. --- If your project

[GitHub] spark issue #13563: [SPARK-15826] [CORE] PipedRDD to strictly use UTF-8 and ...

2016-06-08 Thread sadikovi
Github user sadikovi commented on the issue: https://github.com/apache/spark/pull/13563 @tejasapatil Can you also hard-code UTF-8 for `stderr`? --- 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