[GitHub] flink pull request #6365: [FLINK-9849] [hbase] Hbase upgrade to 2.0.1

[yanghua]

Github user yanghua commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6365#discussion_r203439859
  
    --- Diff: 
flink-connectors/flink-hbase/src/test/java/org/apache/flink/addons/hbase/example/HBaseWriteStreamExample.java
 ---
    @@ -87,22 +90,22 @@ public void configure(Configuration parameters) {
     
                @Override
                public void open(int taskNumber, int numTasks) throws 
IOException {
    -                   table = new HTable(conf, "flinkExample");
    +                   Connection connection = 
ConnectionFactory.createConnection(conf);
    --- End diff --
    
    I think we should add try / catch block to protect the connect leak.


---