Investigate and potentially refactor how to keep extra NodeBearer
The ticket has been renamed - originally it was to investigate how to make work the prepartitioned run for FromCoordsMatching test, but this one finally worked out by the correct cherry-pick of commits in my investigations related to #1837.
However, this should not hide the fact that the code related to this is overly complicated and would gain to be trimmed down and made more explicit - I surmise otherwise we will face many parallel issues while implementing new interpolators.
One of the thing to investigate for instance is the NodeBearer::SetGhost()
overload that is NOT related to a NumberingSubset
- it is still used in one place related to FromCoordsMatching
node bearers, but I'm not convinced it should have stayed.
Likewise, there are several steps that are pretty similar and the naming is not always precise enough - I have renamed a method in #1837 to improve that but there is much more that could be done here.
Overall, as the tests run correctly now the priority has decreased - it is no longer a blocker to the next release - but it's good to keep that in mind nonetheless, so I will let this issue opened.