Kurt Deschler has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17842 )

Change subject: IMPALA-5741: Initial support for reading tiny RDBMS tables
......................................................................


Patch Set 27:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/17842/27//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/17842/27//COMMIT_MSG@13
PS27, Line 13:   - It is not distributed.
Unclear what this actually means. Queries executed on the coordinator?


http://gerrit.cloudera.org:8080/#/c/17842/27//COMMIT_MSG@14
PS27, Line 14:   - Don't support following data types for column data from 
external
Change wording:
From: Don't support queries... Don't support datatypes XYZ.
To: Queries... are not supported. ... XYZ datatypes are not supported.


http://gerrit.cloudera.org:8080/#/c/17842/27//COMMIT_MSG@16
PS27, Line 16:   - Only support binary predicates with operators =, !=, <=, >=,
Will between/in/like planned be supported in the future?


http://gerrit.cloudera.org:8080/#/c/17842/27//COMMIT_MSG@43
PS27, Line 43:
Does this require mincluster restart?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8244e978c7717c6f1452f66f1630b6441392e7d2
Gerrit-Change-Number: 17842
Gerrit-PatchSet: 27
Gerrit-Owner: Fucun Chu <chufu...@hotmail.com>
Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward <gsi...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Fucun Chu <chufu...@hotmail.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Kurt Deschler <kdesc...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Comment-Date: Mon, 09 Oct 2023 15:51:09 +0000
Gerrit-HasComments: Yes

Reply via email to