Version suffixed when using `getResourceKey()`

[MartinBernstorff]

Describe the bug
When defining a view with getResourceKey(), the resourcekey has the form: Communication/1000741880/_history/1

To Reproduce
See repro in history_suffix.py from https://github.com/MartinBernstorff/pathling-history-suffix/

Expected behavior
I'm curious about the design choice here, and might simply be lacking context!

When using delta-lake, I would expect that we're merging onto the resourceKey. But that's definitely not the case (see history_suffix_repeat_import.py). Why do you map getResourceKey() to id_versioned instead of id? :-)

[johngrimes]

I've added a fix for this, we're planning to release this with #2520.

[MartinBernstorff]

Happy to hear it! I'll keep an eye out for the release, so we can update ASAP.

Do you have a rough idea about when it might happen? :-)

[MartinBernstorff]

Oh, and of course feel free to close.

[MartinBernstorff]

Taking a step back, should this also be covered by the SQL on FHIR tests?

https://sql-on-fhir.org/extra/impls.html

[johngrimes]

@MartinBernstorff This has been released in v9.1.0 if you want to try it out.