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 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 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
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
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
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 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
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 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
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 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
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 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 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 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 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 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 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 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 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 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 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 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() {
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 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 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 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
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 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 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 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 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 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
33 matches
Mail list logo