Github user uce commented on the pull request:
https://github.com/apache/incubator-flink/pull/111#issuecomment-54443513
Thanks for adding the documentation. :-)
I've checked the configuration files of the tests and bash scripts. I
didn't check every single file, where you replaced the log4j logger though, but
since everything is compiling and the tests are running, it should be fine (the
test that is failing is unrelated to this PR, see
[FLINK-1077](https://issues.apache.org/jira/browse/FLINK-1077)).
+1 to merge. I like the new logging pattern, which now includes the thread:
```
11:33:11.842 [Nephele Executor Thread 2] INFO
org.apache.flink.runtime.execution.ExecutionStateTransition - JM:
ExecutionState set from RUNNING to FINISHING for task CHAIN DataSource
(TextInputFormat
(file:/var/folders/3_/h2rwvy1541g0x_m9bjpqnhfc0000gn/T/org.apache.flink.test.example)
-> FlatMap (org.apache.flink.example.java.wordcount.WordCount$Tokenizer) ->
Combine(SUM(1)) (3/4)
```
---
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 is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
