[GitHub] flink pull request: [FLINK-2874] Fix Avro getter/setter recognitio...

2015-10-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1252 --- 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 enab

[GitHub] flink pull request: [FLINK-2874] Fix Avro getter/setter recognitio...

2015-10-22 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/1252#issuecomment-150318989 Will merge this PR. --- 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 fe

[GitHub] flink pull request: [FLINK-2874] Fix Avro getter/setter recognitio...

2015-10-22 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/1252#issuecomment-150165291 Looks good to me as well. Should we add this fix to 0.10 RC1 as well? --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] flink pull request: [FLINK-2874] Fix Avro getter/setter recognitio...

2015-10-21 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/1252#issuecomment-149927087 I think it's a good fix. It applies to all POJOs, not just Avro-generated classes, but that should be ok. --- If your project is set up for it, you can reply to this e

[GitHub] flink pull request: [FLINK-2874] Fix Avro getter/setter recognitio...

2015-10-20 Thread ukarlsson
Github user ukarlsson commented on the pull request: https://github.com/apache/flink/pull/1252#issuecomment-149559944 @fhueske @rmetzger Hello, I have added the Jira issue: https://issues.apache.org/jira/browse/FLINK-2874 I have also added unit tests