Skip to content

Commit

Permalink
MAPR-22405: DocumentStream interface in OJAI API need not be parametr…
Browse files Browse the repository at this point in the history
…ized
  • Loading branch information
adityakishore committed Sep 9, 2016
1 parent 869cfbd commit 48d195c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
Expand Up @@ -43,7 +43,6 @@
import org.apache.drill.exec.vector.complex.writer.BaseWriter.MapWriter; import org.apache.drill.exec.vector.complex.writer.BaseWriter.MapWriter;
import org.apache.drill.exec.vector.complex.writer.VarBinaryWriter; import org.apache.drill.exec.vector.complex.writer.VarBinaryWriter;
import org.apache.drill.exec.vector.complex.writer.VarCharWriter; import org.apache.drill.exec.vector.complex.writer.VarCharWriter;
import org.ojai.Document;
import org.ojai.DocumentReader; import org.ojai.DocumentReader;
import org.ojai.DocumentReader.EventType; import org.ojai.DocumentReader.EventType;
import org.ojai.DocumentStream; import org.ojai.DocumentStream;
Expand Down Expand Up @@ -80,7 +79,7 @@ public class MaprDBJsonRecordReader extends AbstractRecordReader {


private DrillBuf buffer; private DrillBuf buffer;


private DocumentStream<Document> documentStream; private DocumentStream documentStream;


private Iterator<DocumentReader> documentReaderIterators; private Iterator<DocumentReader> documentReaderIterators;


Expand Down
Expand Up @@ -147,7 +147,7 @@ public static void createJsonTables() throws IOException {


try (Table table = admin.createTable(TMP_BUSINESS_TABLE); try (Table table = admin.createTable(TMP_BUSINESS_TABLE);
InputStream in = getJsonStream("json/business.json"); InputStream in = getJsonStream("json/business.json");
DocumentStream<Document> stream = Json.newDocumentStream(in)) { DocumentStream stream = Json.newDocumentStream(in)) {
for (Document document : stream) { for (Document document : stream) {
table.insert(document, "business_id"); table.insert(document, "business_id");
} }
Expand Down

0 comments on commit 48d195c

Please sign in to comment.