nickwallen commented on a change in pull request #1458: METRON-2177 Upgrade
Profiler for HBase 2.0.2
URL: https://github.com/apache/metron/pull/1458#discussion_r306321103
##########
File path:
metron-analytics/metron-profiler-spark/src/main/java/org/apache/metron/profiler/spark/function/HBaseWriterFunction.java
##########
@@ -117,8 +131,7 @@ public HBaseWriterFunction(Properties properties) {
if(measurements.size() > 0) {
// open an HBase connection
- Configuration config = HBaseConfiguration.create();
- try (LegacyHBaseClient client = new LegacyHBaseClient(tableProvider,
config, tableName)) {
+ try (HBaseClient client = hBaseClientFactory.create(connectionFactory,
HBaseConfiguration.create(), tableName)) {
Review comment:
Make the Batch Profiler write to Hbase using the new `HbaseClient`
abstraction.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
