Skip to content

Conversation

@dilipbiswal
Copy link
Contributor

Only print the error message to the console for Analysis Exceptions in sql-shell

@dilipbiswal
Copy link
Contributor Author

@marmbrus this is for 1.4.2 branch..

@marmbrus
Copy link
Contributor

ok to test

@SparkQA
Copy link

SparkQA commented Oct 30, 2015

Test build #44678 has finished for PR 9375 at commit ed395be.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the right style: https://cwiki.apache.org/confluence/display/SPARK/Spark+Code+Style+Guide

private[hive] case class CommandProcessorResponseWrapper(
    rc: CommandProcessorResponse,
    cause: Throwable)

@SparkQA
Copy link

SparkQA commented Nov 4, 2015

Test build #45007 has finished for PR 9375 at commit f244688.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dilipbiswal
Copy link
Contributor Author

@marmbrus hello Michael, even though the failure is in CliSuite, i don't think its related to the change as i have only made cosmetic changes in the last commit. I also ran CliSuite in my 142 env with my changes and am able to run it cleanly. Can we retest this please ?

@marmbrus
Copy link
Contributor

marmbrus commented Nov 5, 2015

test this please

@SparkQA
Copy link

SparkQA commented Nov 6, 2015

Test build #45150 has finished for PR 9375 at commit f244688.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@marmbrus
Copy link
Contributor

marmbrus commented Nov 6, 2015

Thanks, merging. Can you close this?

asfgit pushed a commit that referenced this pull request Nov 6, 2015
…ceptions

Only print the error message to the console for Analysis Exceptions in sql-shell

Author: Dilip Biswal <[email protected]>

Closes #9375 from dilipbiswal/spark-11188-v142.
@dilipbiswal dilipbiswal closed this Nov 6, 2015
@dilipbiswal
Copy link
Contributor Author

@marmbrus Thank you very much. Closed the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants