gaurav singh has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21015 )

Change subject: IMPALA-12815: Support timestamp for scan predicates for 
external data source table.
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/21015/4/fe/src/main/java/org/apache/impala/planner/DataSourceScanNode.java
File fe/src/main/java/org/apache/impala/planner/DataSourceScanNode.java:

http://gerrit.cloudera.org:8080/#/c/21015/4/fe/src/main/java/org/apache/impala/planner/DataSourceScanNode.java@148
PS4, Line 148:         return new TColumnValue().setString_val(
> timestamp_val is defined as 16 byte binary array. When calling setTimestamp
The timestamp format used by most databases is ISO 8601.

>From the postgres documentation: The output format of the date/time types can 
>be set to one of the four styles ISO 8601, SQL (Ingres), traditional POSTGRES 
>(Unix date format), or German. The default is the ISO format.

ISO     ISO 8601, SQL standard  1997-12-17 07:37:16-08



--
To view, visit http://gerrit.cloudera.org:8080/21015
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If6ffe672b4027e2cee094cec4f99b9df9308e441
Gerrit-Change-Number: 21015
Gerrit-PatchSet: 4
Gerrit-Owner: gaurav singh <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Reviewer: gaurav singh <[email protected]>
Gerrit-Comment-Date: Thu, 15 Feb 2024 15:48:36 +0000
Gerrit-HasComments: Yes

Reply via email to