Michael Blow has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/3043 )

Change subject: [NO ISSUE][FUN] Code generator alternative
......................................................................


Patch Set 12:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/3043/12/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/PointableHelper.java
File 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/PointableHelper.java:

https://asterix-gerrit.ics.uci.edu/#/c/3043/12/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/PointableHelper.java@215
PS12, Line 215:        // Pointable2
              :         switch (checkMissingOrNull(pointable2)) {
              :             case MISSING:
              :                 setMissing(result);
              :                 return true;
              :             case NULL:
              :                 isMeetNull = true;
              :                 break;
              :         }
              :
> Done.
sure; and yes this is what I was suggesting



--
To view, visit https://asterix-gerrit.ics.uci.edu/3043
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icca2e2128c4b0f2bfd8675655cf5296cbbaeba88
Gerrit-Change-Number: 3043
Gerrit-PatchSet: 12
Gerrit-Owner: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose (1000171)
Gerrit-Reviewer: Dmitry Lychagin <[email protected]>
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Comment-Date: Thu, 18 Apr 2019 12:32:55 +0000
Gerrit-HasComments: Yes

Reply via email to