soumyakanti3578 commented on code in PR #5494: URL: https://github.com/apache/hive/pull/5494#discussion_r1802065459
########## ql/src/test/queries/clientpositive/jdbc_filter_expand_row_operator.q: ########## @@ -0,0 +1,21 @@ +--! qt:database:postgres:q_test_book_table.sql + +CREATE EXTERNAL TABLE book (id int, title varchar(100), author int) +STORED BY 'org.apache.hive.storage.jdbc.JdbcStorageHandler' +TBLPROPERTIES ( + "hive.sql.database.type" = "POSTGRES", + "hive.sql.jdbc.driver" = "org.postgresql.Driver", + "hive.sql.jdbc.url" = "jdbc:postgresql://localhost:5432/qtestDB", + "hive.sql.dbcp.username" = "qtestuser", + "hive.sql.dbcp.password" = "qtestpassword", + "hive.sql.table" = "book"); + +explain cbo +select * from book +where id = 0 or (id = 1 and author = 11) or (id = 2 and author = 22); + +-- non jdbc plan containing ROW, for reference +CREATE EXTERNAL TABLE book1 (id int, title varchar(100), author int); +explain cbo +select * from book1 +where id = 0 or (id = 1 and author = 11) or (id = 2 and author = 22); Review Comment: Yes, it is not really required. I added it to highlight differences in the plan generated for JDBC and non JDBC queries. I have removed it in the latest commit. -- 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. To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org