[GitHub] [calcite] zabetak opened a new pull request #1097: [CALCITE-2902] Improve performance of AbstractRelNode#computeDigest

[GitBox <gi...@apache.org>]

zabetak opened a new pull request #1097: [CALCITE-2902] Improve performance of AbstractRelNode#computeDigest
URL: https://github.com/apache/calcite/pull/1097
 
 
   1. Use StringBuilder instead of PrintWriter for building the digest, which is more efficient to initialize and append to.
   2. Improve (sligthly) the performance of AbstractRelNode#getRelTypeName (used in the digest) by performing:
   a. one lastIndexOf operation instead of potentially two;
   b. char instead of char[] array comparison.
   3. Add micro benchmark with different implementations of AbstractRelNode#getRelTypeName showing the benefit of the adopted implementation.
   4. Improve RelWriter interface with default methods and refactor respective classes.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services