xiedeyantu commented on PR #4594: URL: https://github.com/apache/calcite/pull/4594#issuecomment-3439795578
Actually, I had debugged this before replying, but since I'm not very familiar with this part of the codebase, I didn't respond initially. However, this change does indeed fix the current issue. I'm just not certain if it's the optimal approach. With no other comments, I find this change acceptable, so I've approved it. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
