search

prestodb / rfcs

Presto RFCs
Apache License 2.0
0 stars 22 forks source link

Add RFC for JDBC Join Push down #32

Open Thanzeel-Hassan-IBM opened 1 week ago

Thanzeel-Hassan-IBM commented 1 week ago

Internal review done in this PR : https://github.com/prestodb/rfcs/pull/31

Thanzeel-Hassan-IBM commented 6 days ago

Updating the RFC with the above review comments.. The main idea is to reduce the verbosity. This level of in depth details of the implementation is not required.

I will make the modifications and make it easier to follow.

prestodb-ci commented 5 days ago

Saved that user @Thanzeel-Hassan-IBM is from IBM

Thanzeel-Hassan-IBM commented 4 days ago

Hi @aaneja

I have made some changes to the RFC based on your comments.

Can you pls review it again ? Please Re-review these 2 points, in particular : https://github.com/prestodb/rfcs/pull/32#discussion_r1836269989 https://github.com/prestodb/rfcs/pull/32#discussion_r1836057892