Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/2405 to look at the new patch set (#7). Change subject: [NO ISSUE][EXT] Java UDF framework refactoring ...................................................................... [NO ISSUE][EXT] Java UDF framework refactoring - user model changes: no - storage format changes: no - interface changes: no Details: 1. Separated the JObjects and its subtypes. 2. Add JBuiltinTypes so we don't have to create an object to get JType. 3. Dead code removal. 4. Memory usage optimization in JRecord Serialization. 5. Several fixes about incomplete type implementations in Java UDF and getting JObjects in UDF examples. Change-Id: I3b648191b73fe4aad4f2a6ba1c2066c872fa16a9 --- M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IJObject.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IJRecordAccessor.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IJType.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalFunction.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/JTypeObjectFactory.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/JavaFunctionHelper.java D asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ResultCollector.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/JBuiltinType.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/JObjectAccessors.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/JObjectPointableVisitor.java D asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/JObjectUtil.java D asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/JObjects.java M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/JTypeTag.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/ByteArrayAccessibleDataInputStream.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/ByteArrayAccessibleInputStream.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JBoolean.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JByte.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JCircle.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JDate.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JDateTime.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JDouble.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JDuration.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JFloat.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JInt.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JInterval.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JLine.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JList.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JLong.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JMissing.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JNull.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JObject.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JOrderedList.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JPoint.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JPoint3D.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JPolygon.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JRecord.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JRectangle.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JShort.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JString.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JTime.java A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/JUnorderedList.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/AddHashTagsFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/AddHashTagsInPlaceFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/AllTypesFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/CapitalFinderFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/EchoDelayFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/KeywordsDetectorFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/ParseTweetFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/SumFunction.java M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/UpperCaseFunction.java M asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/serde/ARecordSerializerDeserializer.java 51 files changed, 2,185 insertions(+), 1,910 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/05/2405/7 -- To view, visit https://asterix-gerrit.ics.uci.edu/2405 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I3b648191b73fe4aad4f2a6ba1c2066c872fa16a9 Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Xikui Wang <xkk...@gmail.com> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Till Westmann <ti...@apache.org> Gerrit-Reviewer: abdullah alamoudi <bamou...@gmail.com>