John Spray
John Spray
Erik: - appears to be starting read replicas at LSNs in the middle of a wal record. - either we have to handle this, or explicitly say you can't do...
@MMeent is this change still relevant? It dates from before a bunch of other changes in layer file indices, I believe.
Can you say more about what you're trying to accomplish?
When I ask what you're trying to accomplish, I mean at a higher level than just adding the extension: once you have the extension (I can see you've opened a...
UX notes: - How do I know I'm done? That it's fully drained? (e.g. how should it show up in CLI, admin console) - Must retain the ability to do...
Testing: - `test_ondemand_download_replica` failed with a message from this new code -- but I don't see that test setting an explicit LSN? Pretty weird, seems like it might be activating...
What is the impact? Does the compute just reconnect, or is something worse happening?
Impact: generates errors in logs, but no other impact because what we would have uploaded would have been empty.
When productionizing this, we could consider whether state for part-imported timelines could go in the manifest rather than the index, so that we can keep the index as a structure...
Nice catch, how can we write a test reproducer for this case?