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 nu
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 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 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 feat
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 app
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
e
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 featur
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 add
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 and
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 your
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 aft
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: org
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 yo
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: A
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
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 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 runnin
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 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
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 G
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 feat
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 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 mea
23 matches
Mail list logo