You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by "DashShen (via GitHub)" <gi...@apache.org> on 2024/04/30 03:42:35 UTC
[PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
DashShen opened a new pull request, #3780:
URL: https://github.com/apache/calcite/pull/3780
(no comment)
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2084470716
Can there anything be done to handle the case for 8 or more without creating a class for every case?
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2085841915
The "right' fix is to make ComparableListImpl support the required operations, isn't 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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "DashShen (via GitHub)" <gi...@apache.org>.
DashShen commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2095064456
> If hope that #3781 solves the underlying problem
Yes, that works too.
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu closed pull request #3780: [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception.
URL: https://github.com/apache/calcite/pull/3780
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "DashShen (via GitHub)" <gi...@apache.org>.
DashShen commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2084326377
This bug fix addresses the issue that arises when there are seven parameters.
However, it is theoretically possible to encounter the same problem if the number of elements exceeds eight.
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2098801946
Closing in favor of #3781
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2084322978
Does this fix the bug in general, or just for 6 arguments?
If the bug persists for 7 arguments, what should be done about it?
Your commit should have the same text as the issue and PR.
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "DashShen (via GitHub)" <gi...@apache.org>.
DashShen commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2084493120
> Can there anything be done to handle the case for 8 or more without creating a class for every case?
No more, I didn't find a better way to fix these cases for 8 or more parameters.
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-6376]Filtering CTE of at least 6 columns with QUALIFY operation results in exception. [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu commented on PR #3780:
URL: https://github.com/apache/calcite/pull/3780#issuecomment-2086535842
If hope that #3781 solves the underlying problem
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org