Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/803
Can confirm that with this fix, the previously observed issue (NPE) is no
longer reproducible. Thanks for making this change.
---
If your project is set up for it, you can reply to this email and
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/802
Can confirm that with this fix, query times have improved (details will be
updated in JIRA). Thanks for making this change.
---
If your project is set up for it, you can reply to this email and have
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/813
+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 feature
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/586
+1
Functional and TPC-H SF100 tests from [1] successful. Also verified
building Drill with mapr profile and accessing MapR-DB binary tables using the
maprdb format plugin.
[1] https
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/667
Hey @spanchamiamapr, may be we should create an Apache JIRA for this? Let
me know if you want me to.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/667
DRILL-5065 has been filed to track this issue. Kindly update the issue
title and description accordingly. Thanks!
---
If your project is set up for it, you can reply to this email and have your
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/688
+1 (non-binding). Thanks for making the change.
---
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 user Agirish commented on the issue:
https://github.com/apache/drill/pull/688
@spanchamiamapr, @adityakishore can one of you please review this change?
---
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 Agirish commented on a diff in the pull request:
https://github.com/apache/drill/pull/1033#discussion_r150364581
--- Diff: exec/java-exec/src/main/codegen/includes/parserImpls.ftl ---
@@ -215,11 +215,12 @@ SqlNode SqlDropView() :
/**
* Parses a CTAS or
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/1086
+1. I think this is a welcome change for newbies trying out Drill on
personal systems. For production use, the current defaults were anyway not
always sufficient and had to be updated.
---
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/1011
@paul-rogers, I'll give it a try and update with my findings.
---
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/1011
@arina-ielchiieva, sorry was held-up with something. I've just started on
this - will get back shortly.
---
Github user Agirish commented on the issue:
https://github.com/apache/drill/pull/1011
Looks good!
+1. Getting this into AD 1.13.0 would be great for users.
---
13 matches
Mail list logo