RatulDawar commented on code in PR #14299:
URL: https://github.com/apache/hudi/pull/14299#discussion_r2569945095


##########
hudi-trino-plugin/src/main/java/io/trino/plugin/hudi/HudiModule.java:
##########
@@ -59,7 +58,6 @@ public void configure(Binder binder)
 
         configBinder(binder).bindConfig(HudiConfig.class);
 
-        
binder.bind(boolean.class).annotatedWith(TranslateHiveViews.class).toInstance(false);

Review Comment:
   @yihua So we were having this discussion here ->   [Slack 
thread](https://apache-hudi.slack.com/archives/C4D716NPQ/p1762846511135559) 
   I am open to discuss this. I as a trino user want all the functionality that 
this plugin provides with latest trino version. Which this divergence makes it 
difficult to do so. 
   Anyways we can discuss this parallely after making this compatibilty work.  



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to