[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread dsbos
Github user dsbos commented on a diff in the pull request: https://github.com/apache/drill/pull/93#discussion_r34650500 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -138,34 +155,77 @@ public static DrillConfig create(Properties testConfigur

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread dsbos
Github user dsbos commented on a diff in the pull request: https://github.com/apache/drill/pull/93#discussion_r34649690 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -138,34 +155,77 @@ public static DrillConfig create(Properties testConfigur

[jira] [Created] (DRILL-3498) Drillbit in a restart loop after query parquet file in S3 and the query hangs

2015-07-14 Thread Smart Sun (JIRA)
Smart Sun created DRILL-3498: Summary: Drillbit in a restart loop after query parquet file in S3 and the query hangs Key: DRILL-3498 URL: https://issues.apache.org/jira/browse/DRILL-3498 Project: Apache D

Re: Review Request 33836: DRILL-1950: Parquet pushdown filtering

2015-07-14 Thread Adam Gilmore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33836/ --- (Updated July 15, 2015, 4:41 a.m.) Review request for drill and Jacques Nadeau.

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread dsbos
Github user dsbos commented on a diff in the pull request: https://github.com/apache/drill/pull/93#discussion_r34642779 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -44,26 +46,33 @@ import com.google.common.collect.ImmutableList; i

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread sudheeshkatkam
Github user sudheeshkatkam commented on the pull request: https://github.com/apache/drill/pull/93#issuecomment-121452663 It seems we will be logging too much at DEBUG level (mainly through LogicalOperatorBase and PhysicalOperatorUtil); it will clutter Lilith specially. IMO,

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/93#discussion_r34639451 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -138,34 +155,77 @@ public static DrillConfig create(Properties tes

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread jaltekruse
Github user jaltekruse commented on the pull request: https://github.com/apache/drill/pull/93#issuecomment-121437274 DRILL-2815, DRILL-3496 Adding a link to the related JIRA reports --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Resolved] (DRILL-3030) Foreman hangs trying to cancel non-root fragments

2015-07-14 Thread Sudheesh Katkam (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-3030?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sudheesh Katkam resolved DRILL-3030. Resolution: Cannot Reproduce Please reopen if the issue happens again [~khfaraaz] > Foreman

Re: decoding JsonTypeInfo on StoragePluginConfig

2015-07-14 Thread Parth Chandra
I'm with Jacques on this one. The move from Hadoop 1 to Hadoop 2 was a big issue. I would would be in favor of not changing abstract classes to interfaces unless things are broken. On Mon, Jul 13, 2015 at 5:26 PM, Daniel Barclay wrote: > Jacques Nadeau wrote: > >> ... >> >> (As a side commentar

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/93#discussion_r34634944 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -138,34 +155,77 @@ public static DrillConfig create(Properties tes

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread sudheeshkatkam
Github user sudheeshkatkam commented on a diff in the pull request: https://github.com/apache/drill/pull/93#discussion_r34634834 --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java --- @@ -44,26 +46,33 @@ import com.google.common.collect.ImmutableLis

[GitHub] drill pull request: Enhanced logging and associated code hygiene

2015-07-14 Thread dsbos
GitHub user dsbos opened a pull request: https://github.com/apache/drill/pull/93 Enhanced logging and associated code hygiene You can merge this pull request into a Git repository by running: $ git pull https://github.com/dsbos/incubator-drill bugs/drill-2815-etc-Logging Alt

[GitHub] drill pull request: DRILL-3122: Changing a session option to defau...

2015-07-14 Thread sudheeshkatkam
GitHub user sudheeshkatkam opened a pull request: https://github.com/apache/drill/pull/92 DRILL-3122: Changing a session option to default value results in status as changed @jaltekruse please review. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] drill pull request: DRILL-3497: Throw UserException#validationErro...

2015-07-14 Thread sudheeshkatkam
GitHub user sudheeshkatkam opened a pull request: https://github.com/apache/drill/pull/91 DRILL-3497: Throw UserException#validationError for errors when modifying options Rebased on top of DRILL-2304. @jaltekruse please review the third commit: 6ec7b32 You can merge this

[GitHub] drill pull request: DRILL-2304: Case sensitivity - system and sess...

2015-07-14 Thread sudheeshkatkam
GitHub user sudheeshkatkam opened a pull request: https://github.com/apache/drill/pull/90 DRILL-2304: Case sensitivity - system and session options are case sensitive @jaltekruse please review. You can merge this pull request into a Git repository by running: $ git pull https:/

[jira] [Created] (DRILL-3497) Throw UserException#validationError for errors when modifying options

2015-07-14 Thread Sudheesh Katkam (JIRA)
Sudheesh Katkam created DRILL-3497: -- Summary: Throw UserException#validationError for errors when modifying options Key: DRILL-3497 URL: https://issues.apache.org/jira/browse/DRILL-3497 Project: Apac

[jira] [Created] (DRILL-3496) Augment classpath scanning and DrillConfig logging.

2015-07-14 Thread Daniel Barclay (Drill) (JIRA)
Daniel Barclay (Drill) created DRILL-3496: - Summary: Augment classpath scanning and DrillConfig logging. Key: DRILL-3496 URL: https://issues.apache.org/jira/browse/DRILL-3496 Project: Apache Dr

Re: Review Request 36208: DRILL-3455: If a drillbit, that contains fragments for the current query, dies the QueryManager will fail the query even if those fragments already finished successfully

2015-07-14 Thread Jacques Nadeau
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36208/#review91657 --- Ship it! Looks good. A small improvement on the javadoc might be g

Community hangout will start in 5 minutes

2015-07-14 Thread Abdel Hakim Deneche
Link to the hangout: https://plus.google.com/hangouts/_/event/ci4rdiju8bv04a64efj5fedd0lc Thanks! -- Abdelhakim Deneche Software Engineer Now Available - Free Hadoop On-Demand Training

[GitHub] drill pull request: Drill 3156: Fix Calcite trace by registering R...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/89 --- 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 feature enabled and wishes so, or if the feature is enable