>From Ian Maxon <[email protected]>:

Attention is currently required from: Suryaa Charan Shivakumar.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390 )

Change subject: [ASTERIXDB-3224] Added support for printing logical plan in DOT 
format
......................................................................


Patch Set 5:

(2 comments)

Commit Message:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390/comment/5f42c1e2_78347659
PS2, Line 7: [NO ISSUE] Added support for printing logical plan in DOT format
> also there's an issue for this, so let's add it: https://issues.apache. […]
Done


File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SessionConfig.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390/comment/4c5342b8_3e15a196
PS4, Line 43:     private static final long serialVersionUID = 2L;
> increment this since we changed some enums
after further discussion, according to 
https://docs.oracle.com/javase/8/docs/platform/serialization/spec/version.html#a6678
 , the changes to the class shouldn't require this



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19390
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: Ia6e37080a581292744ddd9030b294936513c15ac
Gerrit-Change-Number: 19390
Gerrit-PatchSet: 5
Gerrit-Owner: Suryaa Charan Shivakumar <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-CC: Ian Maxon <[email protected]>
Gerrit-Attention: Suryaa Charan Shivakumar <[email protected]>
Gerrit-Comment-Date: Fri, 14 Mar 2025 14:35:59 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Ian Maxon <[email protected]>
Comment-In-Reply-To: Suryaa Charan Shivakumar <[email protected]>
Gerrit-MessageType: comment

Reply via email to