[GitHub] flink issue #2850: [FLINK-4895] Drop Hadoop1 support

2016-11-28 Thread rmetzger
Github user rmetzger commented on the issue:

https://github.com/apache/flink/pull/2850
  
Thank you for the review. I'll merge it once travis is green.


---
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 #2850: [FLINK-4895] Drop Hadoop1 support

2016-11-28 Thread StephanEwen
Github user StephanEwen commented on the issue:

https://github.com/apache/flink/pull/2850
  
Looks good to me.

+1 to merge this


---
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 #2850: [FLINK-4895] Drop Hadoop1 support

2016-11-23 Thread rmetzger
Github user rmetzger commented on the issue:

https://github.com/apache/flink/pull/2850
  
Good catch. I removed the reflection magic from the class.


---
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 #2850: [FLINK-4895] Drop Hadoop1 support

2016-11-22 Thread rmetzger
Github user rmetzger commented on the issue:

https://github.com/apache/flink/pull/2850
  
I haven't tried it yet.


---
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 #2850: [FLINK-4895] Drop Hadoop1 support

2016-11-22 Thread greghogan
Github user greghogan commented on the issue:

https://github.com/apache/flink/pull/2850
  
Does Flink build against Hadoop 3.0.0-alpha1?


---
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.
---