impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Feng Guangyuan (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables
Date Wed, 03 Aug 2016 04:28:52 GMT
Feng Guangyuan has posted comments on this change.

Change subject: IMPALA-3622: Unacceptable '\0' used as fields terminator when creating  tables

Patch Set 2:


Yes Jim, I will end it. Thanks for your patient, first.
Indeed, I need your help. I don't know whether my
previous comments have described the bug appearance or not, and where to add tests.
Hope you give me some suggestions, thank you very much.:)
Commit Message:

Line 11: is not allowed to be a delimiter in Hive or Impala, actually. Impala
> This doesn't really explain - can you summarize the bug that occurs, please
According to the official documentation, in IMPALA 1.3.1 and
higher, the delimiter character ‘\0’ is legal
to use the ASCII 0 (null) character for text tables, and a SQL
sample is provided.
  But it fails in practice, running with impala-shell.Also I 
tried ‘\u0000’.After investigated, I found it was translated 
as empty string literal through getUnescapedValueorNull(), due 
to delimiter '\0', i.e. "" as the delimiter to be handled in 
the subsequent internal process.
More details see:

Line 14: 
> until when?
I mean I am not sure this patch has supplied the best approach
to solve this issue,but it's confirmed after you approve it.
File fe/src/main/java/com/cloudera/impala/catalog/

Line 60:     if (fieldDelimiter_ != null && fieldDelimiter_.equals("\0")) return new
> What happens if fieldDelimiter is '\0' and lineDelimiter is '\0'?
ImpalaRuntimeException: Error making 'createTable' RPC to 
Hive Metastore.

More details, please see issue:

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Id67634282ecf0161a3aa7a3d7f8711b49828668e
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Feng Guangyuan <>
Gerrit-Reviewer: Feng Guangyuan <>
Gerrit-Reviewer: Jim Apple <>
Gerrit-HasComments: Yes

View raw message