[GitHub] metron issue #901: METRON-1410 [MPACK] Check for existing HBASE tables befor...

2018-01-26 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/901 Thanks for fixing this @ottobackwards. +1 via inspection from me as well. ---

[GitHub] metron issue #901: METRON-1410 [MPACK] Check for existing HBASE tables befor...

2018-01-25 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/901 I'm going to wait for @lvets a chance to try his scenario ---

[GitHub] metron issue #901: METRON-1410 [MPACK] Check for existing HBASE tables befor...

2018-01-25 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/901 +1 Looks good @ottobackwards. Thanks for fixing this! I have not tested this myself, but it looks solid. Let me know if you'd prefer me to spin this up to get a second test run in.