Paul Rogers created DRILL-5833:
--
Summary: Parquet reader fails with assertion error for Decimal9,
Decimal18 types
Key: DRILL-5833
URL: https://issues.apache.org/jira/browse/DRILL-5833
Project: Apache
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/928
Looks like significant changes where made to the UI since this PR was first
posted. Will need to rework my changes in that area. Please review the other
bits while that rework is in progress.
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/970
@ilooner, please review the option-related changes. @sachouche, can you
review the Parquet parts?
---
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/950
@parthchandra could you please resolve the conflicts?
---
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/928
@paul-rogers could you please resolve the conflicts?
---
Please use this link to join the hangout:
https://plus.google.com/hangouts/_/event/ci4rdiju8bv04a64efj5fedd0lc
Thanks,
Chunhui
Github user parthchandra commented on the issue:
https://github.com/apache/drill/pull/950
Squashed the review changes and rebased on master.
---
GitHub user paul-rogers opened a pull request:
https://github.com/apache/drill/pull/970
DRILL-5832: Migrate OperatorFixture to use SystemOptionManager rather than
mock
The `OperatorFixture` provides structure for testing individual operators
and other "sub-operator" bits of code.
Charles Givre created DRILL-5834:
Summary: Add Networking Functions
Key: DRILL-5834
URL: https://issues.apache.org/jira/browse/DRILL-5834
Project: Apache Drill
Issue Type: Improvement
GitHub user cgivre opened a pull request:
https://github.com/apache/drill/pull/971
Drill-5834 Add Networking Functions
This is a collection of Networking Functions to facilitate network
analysis. The functions include:
- **inet_aton(``)**: Converts an IPv4 address into
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/963
Thanks for the improvements.
LGTM
+1
---
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142573753
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ---
@@ -351,7 +351,35 @@
MAX_QUERY_MEMORY_PER_NODE_KEY, 1024 *
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142576288
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142574038
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142574389
--- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
@@ -240,7 +240,7 @@ drill.exec: {
}
},
sort: {
-
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142577230
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142576399
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142575862
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/928
Rebased onto the latest master.
The profile UI in master had changed since this PR was first posted.
Reworked the query overview information to use the much better style added
recently
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142579551
--- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
@@ -240,7 +240,7 @@ drill.exec: {
}
},
sort: {
-
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142579717
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ---
@@ -351,7 +351,35 @@
MAX_QUERY_MEMORY_PER_NODE_KEY, 1024 *
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142579793
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142579925
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142580052
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142580204
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/960#discussion_r142580136
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/util/MemoryAllocationUtilities.java
---
@@ -56,30 +59,90 @@ public static void
Github user jinfengni commented on the issue:
https://github.com/apache/drill/pull/968
Project push-down rule may not work in the way we want in at least two
cases: 1) there is a bug in the code that we are not aware of, 2) it would not
push the list of columns for "select *", when
Paul Rogers created DRILL-5835:
--
Summary: Deprecate special, hard-coded directories in Drill launch
scripts
Key: DRILL-5835
URL: https://issues.apache.org/jira/browse/DRILL-5835
Project: Apache Drill
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/968
Thanks for the explanation! Bugs in the code are always a concern, but I
personally prefer to fix them rather than write more code that tries to work
around potential, unknown bugs in other code.
29 matches
Mail list logo
