liyafan82 commented on issue #1839: [CALCITE-3836] The hash codes of RelNodes are unreliable URL: https://github.com/apache/calcite/pull/1839#issuecomment-593187160 > If the rel node has the same id, that means it is the same object, since each rel has distinct id. So the default hashcode() serves its purpose. But I don't remember there is any place using RelNode as the hash map keys. @hsyuan Thanks a lot for your feedback. One example that comes to my mind is the HepPlanner. It uses a DefaultDirectedGraph internally, and in the DefaultDirectedGraph, the HepVertex is used as key of the hash map.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
