You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kylin.apache.org by GitBox <gi...@apache.org> on 2019/07/17 03:18:09 UTC

[GitHub] [kylin] hit-lacus edited a comment on issue #747: KYLIN-3832 Query pushdown support postgresql

hit-lacus edited a comment on issue #747: KYLIN-3832 Query pushdown support postgresql
URL: https://github.com/apache/kylin/pull/747#issuecomment-512083217
 
 
   Hi, I think it is a good feature, I know sereval kylin user want to use pg as datasource of Kylin, but they fail to implement their own adaptor, it should be a good feature in next release. By the way, I have two question:
   1. Firstly, have you ever test each **FUNCTION_DEF** in _postgresql.xml_ and make sure they were all translate into correct way?
   2. Seems you indent is not correct in Java file, two whitespaces is not correct.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services