danny0405 commented on issue #1453: [CALCITE-3346] Enable some ignored tests in RelOptRuleTests URL: https://github.com/apache/calcite/pull/1453#issuecomment-535325011 > @danny0405 , what do you think? > Should I change `transitiveInference` back to the old way, without sql parameter, as @julianhyde suggested? If i understand correctly, Julian means to do more refactoring for the method so it looks more concise ? Or we should revert the change @julianhyde ?
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
