[GitHub] [flink] KurtYoung commented on issue #9417: [FLINK-13688][hive] Limit the parallelism/memory of HiveCatalogUseBlinkITCase

2019-08-15 Thread GitBox
KurtYoung commented on issue #9417: [FLINK-13688][hive] Limit the parallelism/memory of HiveCatalogUseBlinkITCase URL: https://github.com/apache/flink/pull/9417#issuecomment-521855975 Verified locally, +1. This is an automate

[GitHub] [flink] KurtYoung commented on issue #9417: [FLINK-13688][hive] Limit the parallelism/memory of HiveCatalogUseBlinkITCase

2019-08-13 Thread GitBox
KurtYoung commented on issue #9417: [FLINK-13688][hive] Limit the parallelism/memory of HiveCatalogUseBlinkITCase URL: https://github.com/apache/flink/pull/9417#issuecomment-520734657 I would like to repeat my first question: why not just extending ` AbstractTestBase`

[GitHub] [flink] KurtYoung commented on issue #9417: [FLINK-13688][hive] Limit the parallelism/memory of HiveCatalogUseBlinkITCase

2019-08-12 Thread GitBox
KurtYoung commented on issue #9417: [FLINK-13688][hive] Limit the parallelism/memory of HiveCatalogUseBlinkITCase URL: https://github.com/apache/flink/pull/9417#issuecomment-520695972 How about we use a pre configured mini cluster instead of lower the memory request? Because it's not 100%