Cédric Villemain
Cédric Villemain
> > > Thanks for your investigations. The team will look at this in more detail as soon as we get a chance. > > > > > > I've...
> > However, adding my rewrite function just before `PlanRouterQuery()` in `RouterJob()` does the job. > > Nice. > > > Maybe it's possible to push down a bit more...
> > There is very probably a design choice to be made, if possible not by me but Citus committers on what contract Citus want to have with parser tree...
> > Thank you for the feedback @colm-mchugh. > > FYI I am still studying alternative ways to achieve the goal, including hook mentioned. Also a step back to look...
I have added more tested related to this PR and found another problem in `insert select` planner, fixed in https://github.com/citusdata/citus/pull/7912
just pushed some more tests (see indirections.sql) and ERROR on cases not well managed so far.
OK, I have added a list sort function to order target list per paramid value: those are always incremented, with a large increment when switching sublinks, it looks simple enough,...
> @c2main we'd like get this into the upcoming release Citus 13.1, so just want to check if it is possible for you to rebase onto the current main branch,...
> > > That sounds good, there is also this point that we need to WARN/ERROR or have an option to WARN/ERROR when we detect a query which may have...
I created #4522 instead of this comment.