Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1060
@parthchandra , @vrozov
I have done the following modifications:
- Renamed newly added files with the prefix "VL" with "VarLen" as suggested
by @parthchandra
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184114148
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unorderedreceiver/UnorderedReceiverBatch.java
---
@@ -153,8 +153,10 @@ private
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184117938
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unorderedreceiver/UnorderedReceiverBatch.java
---
@@ -182,13 +184,18 @@ public
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184138876
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/record/RawFragmentBatch.java
---
@@ -77,4 +83,46 @@ public long getByteCount
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184112400
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unorderedreceiver/UnorderedReceiverBatch.java
---
@@ -153,8 +153,10 @@ private
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184140733
--- Diff:
exec/memory/base/src/main/java/org/apache/drill/exec/memory/AllocationManager.java
---
@@ -253,10 +261,12 @@ public boolean transferBalance
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1060
Parth,
- I have attached, within the DRILL-5846, two profiles with latest Apache
code and this PR request (bounds checks are off):
o Used one thread in each run
o I observe ~3x
GitHub user sachouche opened a pull request:
https://github.com/apache/drill/pull/1192
DRILL-6299: Fixed a filter pushed down issue when a column doesn't haâ¦
This bug happens when the isNull predicate is applied on a column without
statistics. @arina-ielchiieva can you please
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1060
I have updated this pull request with the following changes:
- Excluded the implicit column optimizations from this pull request (will
be included as part of another Drill Jira)
- Tuned
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1168
I successfully ran the test-suite and some advanced tests which loaded the
jdbc artifacts. Is this enough to validate the fix (and future exclusions to
keep the jar size in-check)? if not, what
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1060
@parthchandra and @vrozov can you please let me know whether you are ok
with the changes.
Thanks!
---
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1170
@parthchandra and @paul-rogers, I have added a comment within the Jira
[DRILL-6223](https://issues.apache.org/jira/browse/DRILL-6223); please let me
know what you think.
Thanks!
---
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1144
I have added a comment within this PR associated JIRA:
[DRILL-6202](https://issues.apache.org/jira/browse/DRILL-6202)
---
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1170
@parthchandra can you also please review this PR?
Thanks!
---
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184197278
--- Diff:
exec/memory/base/src/main/java/org/apache/drill/exec/memory/AllocationManager.java
---
@@ -253,10 +261,12 @@ public boolean transferBalance
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184197379
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unorderedreceiver/UnorderedReceiverBatch.java
---
@@ -153,8 +153,10 @@ private
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184151186
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/unorderedreceiver/UnorderedReceiverBatch.java
---
@@ -182,13 +184,18 @@ public
Github user sachouche commented on a diff in the pull request:
https://github.com/apache/drill/pull/1237#discussion_r184192630
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/record/RawFragmentBatch.java
---
@@ -77,4 +83,46 @@ public long getByteCount
Github user sachouche commented on the issue:
https://github.com/apache/drill/pull/1237
@vrozov, I have implemented the provided suggestions.
---
101 - 119 of 119 matches
Mail list logo