asterixdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "abdullah alamoudi (Code Review)" <>
Subject Change in asterixdb[master]: This change includes the following: 1. allowing short-circui...
Date Wed, 22 Jul 2015 06:23:14 GMT
abdullah alamoudi has posted comments on this change.

Change subject: This change includes the following: 1. allowing short-circuit reads to be
used with external data. 2. enhancing behaviour of index access for Text data. 3. updating
the external data documentation.

Patch Set 7:

File asterix-app/src/main/java/edu/uci/ics/asterix/hyracks/bootstrap/

Line 128:         // why are we not stopping the feed server here?
> Yes, I  believe we should. Please piggyback this change too.
Will do.
File asterix-app/src/test/resources/runtimets/queries/hdfs/hdfs_shortcircuit/hdfs_shortcircuit.1.ddl.aql:

Line 20: (("hdfs"="hdfs://"),("path"="/asterix/textFileS"),("input-format"="sequence-input-format"),("format"="delimited-text"),("delimiter"="."),("local-socket-path"="/var/lib/hadoop-hdfs/dn_socket"));
> what's local-socket-path? It is not intuitive to me :(
the local socket path is a data communications endpoint for exchanging data between the reader
and the DataNode running on the same host. It is used to get information about physical files
corresponding to HDFS blocks so they can be read directly.
File asterix-external-data/src/main/java/edu/uci/ics/asterix/external/adapter/factory/

Line 89: 
> I think, these constants need to live outside this class as these are at a 
That is true. we have them in multiple locations. I will move them to the right place.
File asterix-external-data/src/main/java/edu/uci/ics/asterix/external/indexing/input/

Line 103:         this(in, conf.getInt("io.file.buffer.size", DEFAULT_BUFFER_SIZE));
> Use a constant string
Will do

Line 126:      */
> Awesome comments for method signature :)
Haha. I agree :D

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I041c71391d8704cd800c4446a085beef197e7acf
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Raman Grover <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message