Yingyi Bu has posted comments on this change. Change subject: [ASTERIXDB-1964][FUN] Add type conversion functions ......................................................................
Patch Set 2: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/1861/2/asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml File asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml: PS2, Line 8512: Text Add some negative tests for the error case? 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 I suspect the byte code generation module cannot deal with Lambda expression:-P -- 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
