[GitHub] flink pull request: [FLINK-3334] [conf] Include year-month-day in ...

2016-02-08 Thread StephanEwen
GitHub user StephanEwen opened a pull request: https://github.com/apache/flink/pull/1604 [FLINK-3334] [conf] Include year-month-day in the log output Modifies the date pattern in all log config files You can merge this pull request into a Git repository by running: $ git pull

[GitHub] flink pull request: [FLINK-3334] [conf] Include year-month-day in ...

2016-02-08 Thread mxm
Github user mxm commented on the pull request: https://github.com/apache/flink/pull/1604#issuecomment-181501036 This will make log files harder to read but I see that it is necessary for production systems. --- If your project is set up for it, you can reply to this email and have

[GitHub] flink pull request: [FLINK-3334] [conf] Include year-month-day in ...

2016-02-08 Thread gyfora
Github user gyfora commented on the pull request: https://github.com/apache/flink/pull/1604#issuecomment-181503516 Looks good :) Thanks Stephan! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] flink pull request: [FLINK-3334] [conf] Include year-month-day in ...

2016-02-08 Thread mxm
Github user mxm commented on the pull request: https://github.com/apache/flink/pull/1604#issuecomment-181506978 +1 also from my side. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] flink pull request: [FLINK-3334] [conf] Include year-month-day in ...

2016-02-08 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1604#issuecomment-181510175 Yeah, streaming systems tend to run for longer than a day, so it makes sense to log the day... --- If your project is set up for it, you can reply to this email

[GitHub] flink pull request: [FLINK-3334] [conf] Include year-month-day in ...

2016-02-08 Thread StephanEwen
Github user StephanEwen closed the pull request at: https://github.com/apache/flink/pull/1604 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] flink pull request: [FLINK-3334] [conf] Include year-month-day in ...

2016-02-08 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1604#issuecomment-181614932 Manually merged in d51bec1524a9832a7e6c0f6c5d5be5a42712d365 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub