[HANGOUT] Topics for 9/5/2017

2017-09-04 Thread Boaz Ben-Zvi
We shall have a Drill hangout tomorrow (Tuesday Sept 5) at 10 AM Pacific. Please suggest any topics by replying to this thread or bring them up during the hangout. Hangout link: https://plus.google.com/hangouts/_/event/ci4rdiju8bv04a64efj5fedd0lc Thanks, Boaz

[GitHub] drill issue #930: DRILL-5761: Disable Lilith ClassicMultiplexSocketAppender ...

2017-09-04 Thread jinfengni
Github user jinfengni commented on the issue: https://github.com/apache/drill/pull/930 The proposal from @vrozov makes sense to me. As long as Lilith is not completely disabled, I'm fine with it. ---

[GitHub] drill pull request #931: DRILL-5763: Fix NPE during MapRDBSubScan deserializ...

2017-09-04 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/931#discussion_r136856876 --- Diff: contrib/format-maprdb/src/main/java/org/apache/drill/exec/store/mapr/db/MapRDBSubScan.java --- @@ -71,14 +71,32 @@ public

[GitHub] drill pull request #901: DRILL-5709: Provide a value vector method to conver...

2017-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/901 ---

[GitHub] drill issue #901: DRILL-5709: Provide a value vector method to convert a vec...

2017-09-04 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/901 +1 ---

[GitHub] drill pull request #931: DRILL-5763: Fix NPE during MapRDBSubScan deserializ...

2017-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/931 ---

Re: Disable Lilith SOCKET connection by default.

2017-09-04 Thread Vova Vysotskyi
Vlad Rozov, thanks for the idea to conditionally enable Lilith appender. I agree with you that the order of logback.xml files is undefined. I suppose that these files are used for debugging separate tests when logback config file is set manually (-Dlogback.configurationFile). 2017-09-01 22:49

[jira] [Created] (DRILL-5767) Fix error when querying data with the same column names as implicit columns

2017-09-04 Thread Arina Ielchiieva (JIRA)
Arina Ielchiieva created DRILL-5767: --- Summary: Fix error when querying data with the same column names as implicit columns Key: DRILL-5767 URL: https://issues.apache.org/jira/browse/DRILL-5767

[jira] [Created] (DRILL-5766) Stored XSS in APACHE DRILL

2017-09-04 Thread Sanjog Panda (JIRA)
Sanjog Panda created DRILL-5766: --- Summary: Stored XSS in APACHE DRILL Key: DRILL-5766 URL: https://issues.apache.org/jira/browse/DRILL-5766 Project: Apache Drill Issue Type: Bug

[GitHub] drill issue #930: DRILL-5761: Disable Lilith ClassicMultiplexSocketAppender ...

2017-09-04 Thread vvysotskyi
Github user vvysotskyi commented on the issue: https://github.com/apache/drill/pull/930 I have implemented the proposal of @vrozov to conditionally enable Lilith appender. I have used `ThresholdFilter` in Lilith appender, so it was not needed to change all places where it is used.

[GitHub] drill issue #916: DRILL-5377: Five-digit year dates are displayed incorrectl...

2017-09-04 Thread vdiravka
Github user vdiravka commented on the issue: https://github.com/apache/drill/pull/916 I've Added TODO with reference to [SQLLine dateFormat, timeFormat, timestampFormat](https://github.com/julianhyde/sqlline/issues/66) issue. The branch is rebased to master version @paul-rogers

[GitHub] drill pull request #930: DRILL-5761: Disable Lilith ClassicMultiplexSocketAp...

2017-09-04 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/930#discussion_r136835528 --- Diff: common/src/test/resources/logback.xml --- @@ -16,17 +16,22 @@ 1 true ${LILITH_HOSTNAME:-localhost} +

[GitHub] drill issue #909: DRILL-4264: Allow field names to include dots

2017-09-04 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/909 Thanks @vvysotskyi for the PR and @paul-rogers for reviewing the proposal and code. ---

[GitHub] drill issue #909: DRILL-4264: Allow field names to include dots

2017-09-04 Thread amansinha100
Github user amansinha100 commented on the issue: https://github.com/apache/drill/pull/909 Merged in d105950a7a9fb2ff3acd072ee65a51ef1fca120e. @vvysotskyi pls close the PR (for some reason github is not showing me the option to close). ---