Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-1964][FUN] Add type conversion functions ......................................................................
Patch Set 2: Code-Review+1 (4 comments) Just smallish stuff ... https://asterix-gerrit.ics.uci.edu/#/c/1861/2/asterixdb/asterix-doc/src/main/markdown/builtins/11_type.md File asterixdb/asterix-doc/src/main/markdown/builtins/11_type.md: PS2, Line 272: Please fix the end-of-line whitespace. PS2, Line 347: 0 0.0 PS2, Line 347: 1 1.0 https://asterix-gerrit.ics.uci.edu/#/c/1861/2/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/ToBigIntDescriptor.java File asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/ToBigIntDescriptor.java: PS2, Line 46: new IFunctionDescriptorFactory() { : @Override : public IFunctionDescriptor createFunctionDescriptor() { : return new ToBigIntDescriptor(); : } replace these with a function reference IFunctionDescriptorFactory FACTORY = ToBigIntDescriptor::new; -- To view, visit https://asterix-gerrit.ics.uci.edu/1861 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7cf119d8a5dd172f4ce2402315fabf7db084559c Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry Lychagin <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-HasComments: Yes
