dawidwys commented on a change in pull request #9433: [FLINK-13708]
[table-planner-blink] transformations should be cleared after execution in
blink planner
URL: https://github.com/apache/flink/pull/9433#discussion_r346851348
##########
File path: flink-python/pyflink/table/tests/test_table_environment_api.py
##########
@@ -348,7 +347,7 @@ def test_explain_with_extended(self):
assert isinstance(actual, str)
def test_explain_with_multi_sinks(self):
- t_env = self.t_env
+ t_env = self.bt_env
Review comment:
@dianfu @sunjincheng121 Could you have a quick look at that change? As far
as I understand we introduce a new initialized table environment (bt_env) that
will be used only in this single test so far. Could we either unify it somehow
(maybe via parametrization) or just instantiate the environment separately in
this test? It can be done in a follow PR I think, I did not want it go
unnoticed.
----------------------------------------------------------------
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
