You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by ad...@apache.org on 2016/03/23 21:42:26 UTC
drill git commit: DRILL-4317: Exceptions on SELECT and CTAS with
large CSV files
Repository: drill
Updated Branches:
refs/heads/master 5dbaafbe6 -> 79a3c164c
DRILL-4317: Exceptions on SELECT and CTAS with large CSV files
this closes #432
Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/79a3c164
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/79a3c164
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/79a3c164
Branch: refs/heads/master
Commit: 79a3c164c1df7a5d7a0b82574316b4a0b1c7593e
Parents: 5dbaafb
Author: adeneche <ad...@gmail.com>
Authored: Wed Mar 16 14:47:18 2016 +0100
Committer: adeneche <ad...@gmail.com>
Committed: Wed Mar 23 20:22:56 2016 +0100
----------------------------------------------------------------------
.../org/apache/drill/exec/store/easy/text/compliant/TextInput.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/drill/blob/79a3c164/exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/TextInput.java
----------------------------------------------------------------------
diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/TextInput.java b/exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/TextInput.java
index c99e867..513476f 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/TextInput.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/TextInput.java
@@ -265,7 +265,7 @@ final class TextInput {
if(lineSeparator2 == NULL_BYTE){
// we found a line separator and don't need to consult the next byte.
- length = (int)(m - bStart);
+ length = (int)(m - bStart) + 1; // make sure we include line separator otherwise query may fail (DRILL-4317)
endFound = true;
break;
}else{