Github user yanghua commented on a diff in the pull request:
https://github.com/apache/flink/pull/6365#discussion_r203439890
--- 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.
---
|