Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/5371
Actually , I want do some test under the latest master code.
---
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/5371
Hi, @StephanEwen .Sorry for later.
1. For the first question, I do not have a machine for this test at this
moment and wont test it until I have a linux machine. Can not get a linux
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/5371
Two more questions about this:
1. There is code in the shell scripts that rotates log file each time you
start / stop the cluster, with .0 /.1/.2/etc suffixes to the log files. Have
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/5371
Thanks @StephanEwen review. Yea, You correct. DailyRollingFileAppender
doesn't support MaxFileSize, RollingFileAppender does. I will changed the code
to a correct version and wait for the CI
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/5371
The CI Server reports the following problem below. Apparently the
configurations cause warnings to be printed. Can you have a look at that?
```
Found non-empty .out files:
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/5371
Changed!
---
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/5371
I watched and the Travis error does not relevant to this issue.
---
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/5371
I have changed the code, @StephanEwen @XuMingmin Welcome to review those
codes ~ Thanks. I make the ```MaxFileSize``` to 200MB and storage for 30 days
now.
---
Github user zhangminglei commented on the issue:
https://github.com/apache/flink/pull/5371
Thanks @StephanEwen ~
---
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/5371
Looks good!
---
10 matches
Mail list logo
