vvysotskyi commented on issue #1524: DRILL-6830: Remove Hook.REL_BUILDER_SIMPLIFY handler after use URL: https://github.com/apache/drill/pull/1524#issuecomment-436550227 We can't merge PR if it causes tests failures. This test fails because after closing `Hook` in `SqlConverter.toRel()` method, rel builder is used during optimizing stage. I would recommend moving `Hook` usage into the place, where it will be closed only after finishing planning.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
