[GitHub] drill issue #897: Drill-5703 Added Syntax Highlighting and Limited Autocompl...

2017-08-09 Thread cgivre
Github user cgivre commented on the issue: https://github.com/apache/drill/pull/897 @parthchandra What do you want to fix in the pagination? IMHO, the query page and the storage plugin editor both are in dire need of syntax highlighting. I was going to work on that next. In the

[jira] [Created] (DRILL-5713) Doing joins on tables that share column names in a JDBC store returns incorrect results

2017-08-09 Thread Timothy Farkas (JIRA)
Timothy Farkas created DRILL-5713: - Summary: Doing joins on tables that share column names in a JDBC store returns incorrect results Key: DRILL-5713 URL: https://issues.apache.org/jira/browse/DRILL-5713

[GitHub] drill issue #891: DRILL-5699: Drill Web UI Page Source Has Links To External...

2017-08-09 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/891 +1 --- 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

[GitHub] drill issue #897: Drill-5703 Added Syntax Highlighting and Limited Autocompl...

2017-08-09 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/897 I also feel we need to fix the pagination of results before we make improvements to the editor. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] drill issue #897: Drill-5703 Added Syntax Highlighting and Limited Autocompl...

2017-08-09 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/897 In general, I'm not in favor of including large javascript files into the code base, except where the JS might be getting deprecated or is not freely downloadable from the web. Alternatively, we

[jira] [Created] (DRILL-5712) Update the pom files with dependency exclusions for commons-codec

2017-08-09 Thread Sindhuri Ramanarayan Rayavaram (JIRA)
Sindhuri Ramanarayan Rayavaram created DRILL-5712: - Summary: Update the pom files with dependency exclusions for commons-codec Key: DRILL-5712 URL: https://issues.apache.org/jira/browse/DRILL-5712

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

2017-08-09 Thread paul-rogers
GitHub user paul-rogers opened a pull request: https://github.com/apache/drill/pull/901 DRILL-5709: Provide a value vector method to convert a vector to nullable Please see the DRILL-5709 for an explanation and example. You can merge this pull request into a Git repository by

[GitHub] drill pull request #874: DRILL-5663: Drillbit fails to start when only keyst...

2017-08-09 Thread parthchandra
Github user parthchandra commented on a diff in the pull request: https://github.com/apache/drill/pull/874#discussion_r132323524 --- Diff: distribution/src/resources/drill-override-example.conf --- @@ -93,6 +93,13 @@ drill.exec: { credentials: true } },

[GitHub] drill pull request #899: DRILL-5698: Escape version number period separator,...

2017-08-09 Thread darrenbrien
Github user darrenbrien closed the pull request at: https://github.com/apache/drill/pull/899 --- 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

[jira] [Created] (DRILL-5711) Incorrect operator profiles for queries on json files

2017-08-09 Thread Prasad Nagaraj Subramanya (JIRA)
Prasad Nagaraj Subramanya created DRILL-5711: Summary: Incorrect operator profiles for queries on json files Key: DRILL-5711 URL: https://issues.apache.org/jira/browse/DRILL-5711 Project:

Re: compiling drill 1.11.0 with cdh profile

2017-08-09 Thread yuliya Feldman
Could you click on the following link: [1] or try to wget it from the machine you are building on? As apache-14.pom is NOT found in any of the repos except [1] it tries to download it form there and fails.You may or may not have access to other repos, but [1] does not look like accessible for

RE: compiling drill 1.11.0 with cdh profile

2017-08-09 Thread Kunal Khatua
It should be. If this was a dependency problem (like change in versions currently in the Cloudera repo that is not reflected), that would have shown a different set of problems. I hit an issue with some 'banned' dependencies, which might be due to my environment . > [WARNING] Rule 0:

[GitHub] drill issue #889: DRILL-5691: enhance scalar sub queries checking for the ca...

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/889 @weijietong thanks for code formatting. > I wonder if the enhanced code does not break the current unit test ,it will be ok. Can you please check this? Did all unit tests pass

[GitHub] drill issue #900: DRILL-4735: ConvertCountToDirectScan rule enhancements

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/900 @jinfengni addressed code review comments. Please review when possible. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] drill issue #882: DRILL-4735: ConvertCountToDirectScan rule enhancements

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/882 New PR was opened - #900 --- 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

[GitHub] drill pull request #900: DRILL-4735: ConvertCountToDirectScan rule enhanceme...

2017-08-09 Thread arina-ielchiieva
GitHub user arina-ielchiieva opened a pull request: https://github.com/apache/drill/pull/900 DRILL-4735: ConvertCountToDirectScan rule enhancements 1. ConvertCountToDirectScan rule will be applicable for 2 or more COUNT aggregates. To achieve this DynamicPojoRecordReader was

[GitHub] drill pull request #882: DRILL-4735: ConvertCountToDirectScan rule enhanceme...

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva closed the pull request at: https://github.com/apache/drill/pull/882 --- 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

[GitHub] drill issue #899: DRILL-5698: Escape version number period separator, this c...

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/899 @darrenbrien please close this PR. --- 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

[GitHub] drill issue #890: DRILL-5698 Escape version number period separator, this ca...

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/890 There was not need to create new PR. You could just amend your commit message and use `git push -f`. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] drill issue #890: DRILL-5698 Escape version number period separator, this ca...

2017-08-09 Thread darrenbrien
Github user darrenbrien commented on the issue: https://github.com/apache/drill/pull/890 see https://github.com/apache/drill/pull/899 --- 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

[GitHub] drill pull request #899: DRILL-5698: Escape version number period separator,...

2017-08-09 Thread darrenbrien
GitHub user darrenbrien opened a pull request: https://github.com/apache/drill/pull/899 DRILL-5698: Escape version number period separator, this captures ver… …sion numbers with 4 5 or 6 in them, like 1.8.0_144 You can merge this pull request into a Git repository by running:

[GitHub] drill issue #814: Add clickable localhost URL

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/814 @ebuildy can you please check Bridget changes and close the PR if everything is OK? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] drill issue #890: DRILL-5698 Escape version number period separator, this ca...

2017-08-09 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/890 @darrenbrien Can you please also rename the commit message as well, as Paul have mentioned there is a convention here to have Jira number at the beginning: `DRILL-5698: Escape version

RE: compiling drill 1.11.0 with cdh profile

2017-08-09 Thread Dor Ben Dov
Yulia, I ran 'mvn -U -Dskip.Tests clean install -Pcdh' isn't this enough? What am I missing on cloudera? When I am taking drill to hortonworks, it work well by the way. Regards, Dor -Original Message- From: yuliya Feldman [mailto:yufeld...@yahoo.com.INVALID] Sent: יום ג 08 אוגוסט 2017