Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/824#discussion_r150998084
--- Diff:
metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/HBaseDao.java
---
@@ -138,9 +142,17 @@ private Document getDocumentFromResult(Result result)
throws IOException {
Map.Entry<byte[], byte[]> entry= columns.lastEntry();
Long ts = Bytes.toLong(entry.getKey());
if(entry.getValue()!= null) {
- Map<String, Object> json = JSONUtils.INSTANCE.load(new
String(entry.getValue()), new TypeReference<Map<String, Object>>() {
- });
- return new Document(json, Bytes.toString(result.getRow()), (String)
json.get(SOURCE_TYPE), ts);
+ Map<String, Object> json = JSONUtils.INSTANCE.load(new
String(entry.getValue()),
+ new TypeReference<Map<String, Object>>() {});
+ ByteArrayInputStream baos = new
ByteArrayInputStream(result.getRow());
--- End diff --
Ok, I submitted a
[PR](https://github.com/merrimanr/incubator-metron/pull/34) against your branch
that abstracts it out the key into an object and adds some explicit unit tests
around it.
---
