[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-02-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3149 --- 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 #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98832296 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,144 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98828506 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,144 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98828345 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,144 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98828283 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,83 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98805341 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,144 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98805386 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,144 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98805834 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,196 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98805094 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,144 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98805674 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/TableInputFormat.java --- @@ -41,7 +41,7 @@ /** * {@link InputF

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98805799 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,196 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98804629 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,83 @@ +/* + * Licensed to the

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-31 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98805953 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,196 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-30 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98487401 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-30 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98486712 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-30 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98486522 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-30 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98455947 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-30 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98435113 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-30 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98434792 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-26 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r98145293 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,196 @@ +/* +

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-26 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97956487 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,196 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-26 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97955449 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-26 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97956419 --- Diff: flink-test-utils-parent/flink-test-utils/src/main/java/org/apache/flink/test/util/TestBaseUtils.java --- @@ -464,7 +464,7 @@ protected static File

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-26 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97954282 --- Diff: flink-connectors/flink-hbase/pom.xml --- @@ -236,6 +246,12 @@ under the License. test test-jar

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-26 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97954670 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97936002 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97934537 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* +

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97934510 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* +

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97934502 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* +

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97934495 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,65 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97934428 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,137 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97934446 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,137 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97934406 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,137 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97927186 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97898531 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,137 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97903962 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97897602 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,137 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97903708 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97901774 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,65 @@ +/* + * Licensed to the

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97904966 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,248 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97900030 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,137 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97738061 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97736554 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ex00
Github user ex00 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97734069 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to the A

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97730820 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-25 Thread ex00
Github user ex00 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97730063 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to the A

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97710068 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97709565 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,160 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97709590 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -19,99 +19,113 @@ package

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97709469 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97709482 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97709518 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -22,54 +22,63 @@ import org.apache.fl

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97709503 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -22,54 +22,63 @@ import org.apache.fl

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97709488 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97702519 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,160 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97702388 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,160 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97701834 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,160 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97701594 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -19,99 +19,113 @@ package org

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97698410 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97700808 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -22,54 +22,63 @@ import org.apache.flink

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97696842 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97696231 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSchema.java --- @@ -0,0 +1,135 @@ +/* + * Licensed to th

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-24 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97699488 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -22,54 +22,63 @@ import org.apache.flink

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-23 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97304540 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-20 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97074517 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-20 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97073974 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-20 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97067569 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-20 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97055300 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-20 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97031401 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-20 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97030686 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-19 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97015379 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-19 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97015321 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-19 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r97015267 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-19 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96824021 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-19 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96643785 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSource.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-19 Thread wuchong
Github user wuchong commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96823925 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96802576 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96790253 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96790136 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread ramkrish86
Github user ramkrish86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96783534 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,117 @@ +/* +

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r9157 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,117 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96668950 --- Diff: flink-connectors/flink-hbase/src/main/java/org/apache/flink/addons/hbase/HBaseTableSourceInputFormat.java --- @@ -0,0 +1,322 @@ +/* + * Lic

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96665731 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,117 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96665689 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,117 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread tonycox
Github user tonycox commented on a diff in the pull request: https://github.com/apache/flink/pull/3149#discussion_r96665755 --- Diff: flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseTableSourceITCase.java --- @@ -0,0 +1,117 @@ +/* + *

[GitHub] flink pull request #3149: FLINK-2168 Add HBaseTableSource

2017-01-18 Thread ramkrish86
GitHub user ramkrish86 opened a pull request: https://github.com/apache/flink/pull/3149 FLINK-2168 Add HBaseTableSource Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration. If your changes take all o