Skip to content

[SQL][TRIVIAL] Lower parser log level to debug#18006

Closed
hvanhovell wants to merge 1 commit intoapache:masterfrom
hvanhovell:lower_parser_log_level
Closed

[SQL][TRIVIAL] Lower parser log level to debug#18006
hvanhovell wants to merge 1 commit intoapache:masterfrom
hvanhovell:lower_parser_log_level

Conversation

@hvanhovell
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Currently the parser logs the query it is parsing at info level. This is too high, this PR lowers the log level to debug.

How was this patch tested?

Existing tests.

Copy link
Copy Markdown
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

LGTM

@SparkQA
Copy link
Copy Markdown

SparkQA commented May 16, 2017

Test build #76981 has finished for PR 18006 at commit 58ef755.

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

@gatorsmile
Copy link
Copy Markdown
Member

Thanks! Merging to master/2.2

@asfgit asfgit closed this in 69bb771 May 16, 2017
robert3005 pushed a commit to palantir/spark that referenced this pull request May 19, 2017
## What changes were proposed in this pull request?
Currently the parser logs the query it is parsing at `info` level. This is too high, this PR lowers the log level to `debug`.

## How was this patch tested?
Existing tests.

Author: Herman van Hovell <hvanhovell@databricks.com>

Closes apache#18006 from hvanhovell/lower_parser_log_level.
lycplus pushed a commit to lycplus/spark that referenced this pull request May 24, 2017
## What changes were proposed in this pull request?
Currently the parser logs the query it is parsing at `info` level. This is too high, this PR lowers the log level to `debug`.

## How was this patch tested?
Existing tests.

Author: Herman van Hovell <hvanhovell@databricks.com>

Closes apache#18006 from hvanhovell/lower_parser_log_level.
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