[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52476961 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,107 @@ +/** + * Licensed

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52465362 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,107 @@ +/** + * Licensed

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52455844 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,107 @@ +/** + * Licensed

Re: Unresolved dependencies

2016-02-10 Thread Abdel Hakim Deneche
Are you using maven ? what error message are you seeing ? On Wed, Feb 10, 2016 at 4:39 AM, Vitalii Diravka wrote: > I saw unresolved dependencies in > /drill/contrib/storage-hive/core/src/test/java/org/apache/drill/exec/ > in the latest version

Re: Unresolved dependencies

2016-02-10 Thread Vitalii Diravka
I use IntelliJ IDEA 2016-02-10 19:38 GMT+02:00 Abdel Hakim Deneche : > Are you using Eclipse or IntelliJ ? > > On Wed, Feb 10, 2016 at 9:29 AM, Vitalii Diravka < > vitalii.dira...@gmail.com> > wrote: > > > Yes, I am. Version of maven is 3.3.3. > > No error. "mvn clean

Re: Unresolved dependencies

2016-02-10 Thread Vitalii Diravka
Yes, I am. Version of maven is 3.3.3. No error. "mvn clean install -DskipTests" is built successfully. But import statements have missing classes (in HiveTestUDFImpls, TestSqlStdBasedAuthorization, TestStorageBasedHiveAuthorization classes). Looks like some library is missing in

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52494516 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,107 @@ +/** + * Licensed

Re: Unresolved dependencies

2016-02-10 Thread Abdel Hakim Deneche
Are you using Eclipse or IntelliJ ? On Wed, Feb 10, 2016 at 9:29 AM, Vitalii Diravka wrote: > Yes, I am. Version of maven is 3.3.3. > No error. "mvn clean install -DskipTests" is built successfully. > But import statements have missing classes (in HiveTestUDFImpls, >

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52494612 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,107 @@ +/** + * Licensed

Re: Unresolved dependencies

2016-02-10 Thread Sudheesh Katkam
I have the same issue. Sometimes, in IntelliJ, I have to manually add the Hive jars to the class path (press option + return on Mac). And sometimes, the hive subproject pom file changes, which I do not commit. I have not investigated what the underlying issue is. > On Feb 10, 2016, at 9:39 AM,

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52498045 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,107 @@ +/** + * Licensed

Re: Unresolved dependencies

2016-02-10 Thread Vitalii Diravka
Alt+Enter (on linux) gives a new maven dependency: org.apache.hive hive-exec 1.2.1 test and after this import statements aren't highlighted. But all hive tests aren't running after that! 2016-02-10 19:53 GMT+02:00 Sudheesh Katkam : > I have the

[GitHub] drill pull request: DRILL-4287: During initial DrillTable creation...

2016-02-10 Thread adeneche
Github user adeneche commented on a diff in the pull request: https://github.com/apache/drill/pull/345#discussion_r52516038 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -182,10 +183,24 @@ public ParquetGroupScan( //

[GitHub] drill pull request: DRILL-4287: During initial DrillTable creation...

2016-02-10 Thread adeneche
Github user adeneche commented on a diff in the pull request: https://github.com/apache/drill/pull/345#discussion_r52519003 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -521,6 +543,25 @@ public void

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52539112 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,107 @@ +/** + * Licensed

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52540911 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -485,12 +486,14 @@ public void

[GitHub] drill pull request: DRILL-4382: Remove dependency on drill-logical...

2016-02-10 Thread StevenMPhillips
GitHub user StevenMPhillips opened a pull request: https://github.com/apache/drill/pull/373 DRILL-4382: Remove dependency on drill-logical from vector package You can merge this pull request into a Git repository by running: $ git pull https://github.com/StevenMPhillips/drill

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52564032 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,108 @@ +/** + * Licensed

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52565738 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -485,12 +486,14 @@ public void

[GitHub] drill pull request: DRILL-4382: Remove dependency on drill-logical...

2016-02-10 Thread StevenMPhillips
Github user StevenMPhillips commented on the pull request: https://github.com/apache/drill/pull/373#issuecomment-182692303 There are two main changes made in this commit: 1. Removed SchemaPath from MaterializedField. Now, a MaterializedField contains a name (String), and a

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52563417 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -791,6 +799,43 @@ public FileGroupScan

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52563924 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -791,6 +799,43 @@ public FileGroupScan

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jinfengni
Github user jinfengni commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52563951 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractGroupScan.java --- @@ -128,4 +128,12 @@ public int getOperatorType() {

Unresolved dependencies

2016-02-10 Thread Vitalii Diravka
I saw unresolved dependencies in /drill/contrib/storage-hive/core/src/test/java/org/apache/drill/exec/ in the latest version github.com/apache/drill. Which library must be added to pom.xml?

[GitHub] drill pull request: DRILL-4020: The not-equal operator returns inc...

2016-02-10 Thread nagix
Github user nagix commented on the pull request: https://github.com/apache/drill/pull/309#issuecomment-182678346 Can anyone merge this? --- 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: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52560352 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -791,6 +799,43 @@ public FileGroupScan

[GitHub] drill pull request: DRILL-4382: Remove dependency on drill-logical...

2016-02-10 Thread jacques-n
Github user jacques-n commented on the pull request: https://github.com/apache/drill/pull/373#issuecomment-182681649 @StevenMPhillips, can you please provide a narrative overview of the nature of these changes? --- If your project is set up for it, you can reply to this email and

[jira] [Created] (DRILL-4382) Remove dependency on drill-logical from vector submodule

2016-02-10 Thread Steven Phillips (JIRA)
Steven Phillips created DRILL-4382: -- Summary: Remove dependency on drill-logical from vector submodule Key: DRILL-4382 URL: https://issues.apache.org/jira/browse/DRILL-4382 Project: Apache Drill

[GitHub] drill pull request: DRILL-4275: Refactor e/pstore interfaces and t...

2016-02-10 Thread hnfgns
Github user hnfgns closed the pull request at: https://github.com/apache/drill/pull/325 --- 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] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52557262 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractGroupScan.java --- @@ -128,4 +128,12 @@ public int getOperatorType()

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52557443 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushLimitToScanRule.java --- @@ -0,0 +1,108 @@ +/** + *

[GitHub] drill pull request: DRILL-4275: create TransientStore for short-li...

2016-02-10 Thread hnfgns
GitHub user hnfgns opened a pull request: https://github.com/apache/drill/pull/374 DRILL-4275: create TransientStore for short-lived objects; refactor PersistentStore to introduce pagination mechanism ps: removed PR#395 mistakenly so starting over. collections/

[GitHub] drill pull request: DRILL-4363: Row count based pruning for parque...

2016-02-10 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/371#discussion_r52558347 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetGroupScan.java --- @@ -791,6 +799,43 @@ public FileGroupScan