[GitHub] drill issue #907: DRILL-5697: Improve performance of filter operator for pat...

2017-08-24 Thread ppadma
Github user ppadma commented on the issue: https://github.com/apache/drill/pull/907 @kkhatua Kunal, we can add more patterns later if we want. For now, let us get the most simple cases done first. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] drill issue #907: DRILL-5697: Improve performance of filter operator for pat...

2017-08-24 Thread ppadma
Github user ppadma commented on the issue: https://github.com/apache/drill/pull/907 @paul-rogers Paul, thanks a lot for the review. I made changes as per your comments. Please review updated diffs. --- If your project is set up for it, you can reply to this email and have your

[GitHub] drill pull request #913: - DRILL-5729 Fix Travis Build

2017-08-24 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/913#discussion_r135148431 --- Diff: .travis.yml --- @@ -13,8 +13,10 @@ # See the License for the specific language governing permissions and # limitations under the License.

[GitHub] drill pull request #913: - DRILL-5729 Fix Travis Build

2017-08-24 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/913#discussion_r135143313 --- Diff: .travis.yml --- @@ -13,8 +13,10 @@ # See the License for the specific language governing permissions and # limitations under the License.

[GitHub] drill issue #913: - DRILL-5729 Fix Travis Build

2017-08-24 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/913 @ilooner-mapr I tested it on my branch and it works with the default jdk and without *sudo*. Add `MAVEN_OPTS="-Xms1G -Xmx1G"` before `mvn`. Can you remove leading dash from the commit message

[GitHub] drill issue #913: - DRILL-5729 Fix Travis Build

2017-08-24 Thread ilooner-mapr
Github user ilooner-mapr commented on the issue: https://github.com/apache/drill/pull/913 @vrozov I got it working with Trusty and explicitly configuring openjdk7. One thing to note is that Trusty was running out of memory doing the build with the default 4gb container size. So I had

[GitHub] drill issue #913: - DRILL-5729 Fix Travis Build

2017-08-24 Thread ilooner-mapr
Github user ilooner-mapr commented on the issue: https://github.com/apache/drill/pull/913 @vrozov It looks like the default jdk for Trusty on travis is jdk8. I'll check to see if explicitly telling travis to use jdk7 with Trusty is also a possible fix. --- If your project is set up

[GitHub] drill issue #913: - DRILL-5729 Fix Travis Build

2017-08-24 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/913 @ilooner-mapr Do you know why it fails on the Trusty (new default)? Going back to using Precise is OK for a while, but my guess that it will eventually become obsolete. --- If your project is set up

Re: Drill 2.0 (design) hackathon

2017-08-24 Thread Aman Sinha
Hi Charles, yes, it would be great if remote folks could participate..I will look into the options for livestreaming. On Thu, Aug 24, 2017 at 8:42 AM, Charles Givre wrote: > Hi Aman, > Would you consider doing some sort of livestream so that those of us who > couldn’t be

Re: Drill 2.0 (design) hackathon

2017-08-24 Thread Charles Givre
Hi Aman, Would you consider doing some sort of livestream so that those of us who couldn’t be there in person can participate? Thanks, — C > On Aug 24, 2017, at 11:39, Aman Sinha wrote: > > Drill Developers, > > In order to kick-start the Drill 2.0 release discussions,

Drill 2.0 (design) hackathon

2017-08-24 Thread Aman Sinha
Drill Developers, In order to kick-start the Drill 2.0 release discussions, I would like to propose a Drill 2.0 (design) hackathon (a.k.a Drill Developer Day ™ J ). As I mentioned in the hangout on Tuesday, MapR has offered to host it on Sept 18th at their offices at 350 Holger Way, San Jose.

[GitHub] drill issue #904: DRILL-5717: change some date time test cases with specific...

2017-08-24 Thread weijietong
Github user weijietong commented on the issue: https://github.com/apache/drill/pull/904 @vvysotskyi please review the update ones --- 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