[GitHub] flink issue #3235: FLINK-5517 Upgrade hbase version to 1.3.0

2017-02-08 Thread tzulitai
Github user tzulitai commented on the issue:

https://github.com/apache/flink/pull/3235
  
Merging to `master`


---
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 #3235: FLINK-5517 Upgrade hbase version to 1.3.0

2017-02-08 Thread rmetzger
Github user rmetzger commented on the issue:

https://github.com/apache/flink/pull/3235
  
Sounds good, thank you for testing it!

+1 to merge then


---
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 #3235: FLINK-5517 Upgrade hbase version to 1.3.0

2017-02-08 Thread tzulitai
Github user tzulitai commented on the issue:

https://github.com/apache/flink/pull/3235
  
From the Maven dependency tree, there seems to be no additional 
dependencies for 1.3.0. Quick check on the compile dependencies of 1.3.0 and 
original 1.2.3 in Maven central, they're identical.

Tested against HBase 1.3.0 with code in 
`org.apache.flink.addons.hbase.example.HBaseWriteExample`, both in IDE and 
cluster submission. No errors were observed.

I think it's a safe upgrade.


---
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 #3235: FLINK-5517 Upgrade hbase version to 1.3.0

2017-02-07 Thread tzulitai
Github user tzulitai commented on the issue:

https://github.com/apache/flink/pull/3235
  
I can look into this tomorrow and check.


---
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 #3235: FLINK-5517 Upgrade hbase version to 1.3.0

2017-02-07 Thread rmetzger
Github user rmetzger commented on the issue:

https://github.com/apache/flink/pull/3235
  
Has anybody of you tested flink against hbase with the 1.3.0 version?
Is Hbase 1.3.0 adding any new dependencies?
I'm asking because its a major version change, which is likely to cause 
some issues.


---
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 #3235: FLINK-5517 Upgrade hbase version to 1.3.0

2017-02-01 Thread tzulitai
Github user tzulitai commented on the issue:

https://github.com/apache/flink/pull/3235
  
Thank you for looking into this @tedyu.

In the 
ML(http://apache-flink-user-mailing-list-archive.2336050.n4.nabble.com/Help-using-HBase-with-Flink-1-1-4-td11094.html),
 Giuliano has also reported that the upgrade fixes the problem.

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