>From Ian Maxon <[email protected]>:

Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364 )

Change subject: [ASTERIXDB-3034][RT] Fenced UDFs
......................................................................


Patch Set 8:

(4 comments)

> Patch Set 7:
>
> (4 comments)

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
File 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java@1488
PS7, Line 1488:             case "podman":
> should reset this file, not used
Done


https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
File 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java@69
PS7, Line 69:     testExecutor = executor;
> this should call the new signature with false
Done


https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/StandardUTF8ToModifiedUTF8DataOutput.java
File 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/StandardUTF8ToModifiedUTF8DataOutput.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/StandardUTF8ToModifiedUTF8DataOutput.java@62
PS7, Line 62:       while (numOfChars > 0 && length < len) {
> reset
Done


https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/hyracks-fullstack/pom.xml
File hyracks-fullstack/pom.xml:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364/7/hyracks-fullstack/pom.xml@658
PS7, Line 658:   </plugin>
> reset this file
Done



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/16364
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ibeb6228f2dc8edbf642e61cd5633c71913e18972
Gerrit-Change-Number: 16364
Gerrit-PatchSet: 8
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Comment-Date: Wed, 01 Jun 2022 00:05:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Ian Maxon <[email protected]>
Gerrit-MessageType: comment

Reply via email to