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 (#3).

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
D 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/RuntimeExternalFunctionUtil.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
50 files changed, 2,212 insertions(+), 2,014 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/05/2405/3
-- 
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: 3
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>

Reply via email to