Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1789: fix an accessor that is used by Hash Table ......................................................................
ASTERIXDB-1789: fix an accessor that is used by Hash Table - Fix an accesor that is used by SerializableHashTable class. This accessor class is required to calculate the original hash value of a tuple when compacting the hash table. Change-Id: I74b6de8f189e841c16735dbd63f13892bbd61c20 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1734 Reviewed-by: Yingyi Bu <buyin...@gmail.com> Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu> BAD: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu> --- R asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/queries/q7.sqlpp M hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/buffermanager/TupleInFrameListAccessor.java 2 files changed, 3 insertions(+), 4 deletions(-) Approvals: Yingyi Bu: Looks good to me, approved Jenkins: Verified; No violations found; No violations found; Verified diff --git a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/queries/q7.sqlpp.dis b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/queries/q7.sqlpp similarity index 100% rename from asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/queries/q7.sqlpp.dis rename to asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/queries/q7.sqlpp diff --git a/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/buffermanager/TupleInFrameListAccessor.java b/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/buffermanager/TupleInFrameListAccessor.java index 2c9d24a..aadcb26 100644 --- a/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/buffermanager/TupleInFrameListAccessor.java +++ b/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/buffermanager/TupleInFrameListAccessor.java @@ -24,8 +24,7 @@ import org.apache.hyracks.api.comm.IFrameTupleAccessor; import org.apache.hyracks.api.dataflow.value.RecordDescriptor; -import org.apache.hyracks.dataflow.std.sort.util.DeletableFrameTupleAppender; -import org.apache.hyracks.dataflow.std.sort.util.IAppendDeletableFrameTupleAccessor; +import org.apache.hyracks.dataflow.common.comm.io.FrameTupleAccessor; import org.apache.hyracks.dataflow.std.structures.TuplePointer; /** @@ -33,11 +32,11 @@ */ public class TupleInFrameListAccessor extends AbstractTuplePointerAccessor { - private IAppendDeletableFrameTupleAccessor bufferAccessor; + private IFrameTupleAccessor bufferAccessor; private List<ByteBuffer> bufferFrames; public TupleInFrameListAccessor(RecordDescriptor rd, List<ByteBuffer> bufferFrames) { - bufferAccessor = new DeletableFrameTupleAppender(rd); + bufferAccessor = new FrameTupleAccessor(rd); this.bufferFrames = bufferFrames; } -- To view, visit https://asterix-gerrit.ics.uci.edu/1734 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74b6de8f189e841c16735dbd63f13892bbd61c20 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Taewoo Kim <wangs...@gmail.com> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Taewoo Kim <wangs...@gmail.com> Gerrit-Reviewer: Yingyi Bu <buyin...@gmail.com>