You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Xikui Wang (Code Review)" <do...@asterixdb.incubator.apache.org> on 2018/03/01 16:58:17 UTC
Change in asterixdb[master]: [NO ISSUE][EXT] Java UDF framework refactoring
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 (#5).
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/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
M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/JObjectAccessors.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
C 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/JOrderedList.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/JUnorderedList.java
R asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/IJType.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JBoolean.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JBuiltinType.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JByte.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JCircle.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JDate.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JDateTime.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JDouble.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JDuration.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JFloat.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JInt.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JInterval.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JLine.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JList.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JLong.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JMissing.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JNull.java
C asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JObject.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JPoint.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JPoint3D.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JPolygon.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JRectangle.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JShort.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JString.java
A asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/java/base/builtin/JTime.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/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
M hyracks-fullstack/algebricks/algebricks-core/pom.xml
M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/AssignPOperator.java
51 files changed, 2,218 insertions(+), 1,914 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/05/2405/5
--
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: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang <xk...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>