[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-15 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-760755775 It looks like the build was successful This is an automated message from the Apache Git Service. To respond to

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-14 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-760115634 Hi,@wuchong ,Test - libraries error,How to fix? This is an automated message from the Apache Git Service. To re

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-14 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-760015927 > The compile is failed again... fix again This is an automated message from the Apache Git Service. To

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-13 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-759975195 The conflict has been resolved @wuchong This is an automated message from the Apache Git Service. To respond t

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-13 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-759966071 ok,a piece of cake This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-13 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-759955407 Thanks for review, @wuchong This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-13 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-759902228 > The following case is failed: > > ``` > [ERROR] Tests run: 11, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 13.404 s <<< FAILURE! - in org.apache.flink.table.c

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-13 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-759412589 @wuchong ,Thanks for "setup" doc This is an automated message from the Apache Git Service. To respond to the

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-12 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-759165382 @wuchong ,Jark, I hava resovle the conflicts. This is an automated message from the Apache Git Service. To resp

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-12 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-758602392 cc,@godfreyhe This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [flink] xiaoHoly commented on pull request #14616: [FLINK-20879][table-api] Use MemorySize type instead of String type for memory ConfigOption in ExecutionConfigOptions

2021-01-12 Thread GitBox
xiaoHoly commented on pull request #14616: URL: https://github.com/apache/flink/pull/14616#issuecomment-758527007 cc,@wuchong .please review this change,thanks This is an automated message from the Apache Git Service. To resp