[GitHub] flink issue #3713: [FLINK-6299] make all IT cases extend from TestLogger

2017-04-15 Thread zentol
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/3713
  
merging.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink issue #3713: [FLINK-6299] make all IT cases extend from TestLogger

2017-04-13 Thread NicoK
Github user NicoK commented on the issue:

https://github.com/apache/flink/pull/3713
  
actually, the error was in the flink-hbase `pom.xml` - please let me 
include these hotfixes here as well which fix the actual error and prevent 
future errors.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink issue #3713: [FLINK-6299] make all IT cases extend from TestLogger

2017-04-13 Thread NicoK
Github user NicoK commented on the issue:

https://github.com/apache/flink/pull/3713
  
Strangely, this does not yet provide logging for all test cases. After also 
applying #3708, `HBaseConnectorITCase`, for example does show up in the 
`flink-fast-tests-a` standard output but neither in `1.log` nor `2.log`...
let me test this further


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink issue #3713: [FLINK-6299] make all IT cases extend from TestLogger

2017-04-12 Thread zentol
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/3713
  
Looks good, +1 to merge.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---