Chris Siefert

@egboman Ignore the Tpetra stuff, but are the Zoltan2 changes reasonable? This is Option #2 (I couldn't see a good way of making Option #1 work when I tried) https://github.com/trilinos/Trilinos/tree/csiefer-tpetra-zoltan2-reduced-eti

So is this a "deprecate" or a "just rename stuff" situation?

Those PRs should fix all of the Tpetra-relevant portions of this issue.

@fnrizzi I'm hoping https://github.com/trilinos/Trilinos/pull/11946 will address this.

@fnrizzi Apologies for misreading your issue. I'll take a closer look.

@fnrizzi My first attempt to translate this into a unit test produced the correct answer (75 instead of 1). I'm going to go try your example against an installed trilinos...

@fnrizzi When I your exact code against installed Trilinos, I get: ``` starting case that works CONSTR-block-A ext = 15 CONSTR-tpetr-A ext = 75 CONSTR-block-B ext = 15 CONSTR-tpetr-B ext...

I got the above using my VOTD checkout of develop. And now my unit test code, cut and paste into an external example, generates the results I see above. Weird.

As per my discussions with @cwpearson, it appears that BlockVector does not std::move correctly and that this may be the root of your issue.

@brian-kelley The test basically points to old stack variables, so valgrind isn't helpful