xiedeyantu commented on PR #4878: URL: https://github.com/apache/calcite/pull/4878#issuecomment-4232891727
> So there is no doubt: > > -1 > > Three of us (me and Steven in Jira, Alessandro in the PR comments) have expressed doubts that this is even a bug. > > Let's not merge any fix until we decide what, if anything, needs to be fixed. @julianhyde Of course, I won't merge it before receiving approval from the committer. The conversation in the JIRA hasn't progressed for several days, so I assumed everyone agreed with this conservative approach. I would also like to thank Julian for the timely reminder. Perhaps I misunderstood. However, I agree with the solution described by Alessandro. This PR can be put on hold for now. Shall we move the discussion back to the JIRA? -- 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]
