Re:Re: CatalogTestBase.* failed on travis

2019-06-20 Thread Haibo Sun
Thanks for the quick fix. Best, Haibo 在 2019-06-20 21:24:47,"jincheng sun" 写道: >Hi Haibo, > >The issue is fixed, great thanks for your timely report on this issue! > >Best, >Jincheng > > >jincheng sun 于2019年6月20日周四 下午9:04写道: > >> Hi flink devs: >> >> Normally, in order not to affect the merge

Re: CatalogTestBase.* failed on travis

2019-06-20 Thread jincheng sun
Hi Haibo, The issue is fixed, great thanks for your timely report on this issue! Best, Jincheng jincheng sun 于2019年6月20日周四 下午9:04写道: > Hi flink devs: > > Normally, in order not to affect the merge of the followup PR, we need to > revert the problematic commit, but since Dian quickly opens the

Re: CatalogTestBase.* failed on travis

2019-06-20 Thread jincheng sun
Hi flink devs: Normally, in order not to affect the merge of the followup PR, we need to revert the problematic commit, but since Dian quickly opens the hotfix PR, this time we take the hotfix. So great thanks to Dian! Hi Dian, you are right,the test fail caused by the PR #8786, and I completely

Re: CatalogTestBase.* failed on travis

2019-06-20 Thread jincheng sun
Great thanks for the help to fix it Dian! I'll merge it. Dian Fu 于2019年6月20日周四 下午7:50写道: > Hi Haibo, > > Thanks a lot for report this bug. I guess it's caused by this PR: > https://github.com/apache/flink/pull/8786 < > https://github.com/apache/flink/pull/8786> @Bowen. I think we'd better > m

Re: CatalogTestBase.* failed on travis

2019-06-20 Thread Dian Fu
Hi Haibo, Thanks a lot for report this bug. I guess it's caused by this PR: https://github.com/apache/flink/pull/8786 @Bowen. I think we'd better merge the code ONLY after the travis passed, especially when the changes are not just hotfix/documentati

CatalogTestBase.* failed on travis

2019-06-20 Thread Haibo Sun
Hi, guys I noticed that all of the Travis tests reported a number of failures as following. Is anyone working on this problem? __ CatalogTestBase.test_table_exists ___ self = def test_table_exists(self): > self.catalog.create_database(self.