nickva commented on issue #5867: URL: https://github.com/apache/couchdb/issues/5867#issuecomment-3807098962
> and there's a backward compatibility issue. I think both proposal would be a compatibility issue. But it also feels more a bug fix. The replication always restarting from the since_seq parameter and overwriting the checkpoints again seems broken and not useful or intuitive. > the replication checkpoint should supercede the since_seq value (it's a "replication that started at this seq" not "a replication that must always start at this seq"). Which for safety means the since_seq value has to be included in the replication hash (its presence means it's a different replication from one that is identical in all properties except it lacks since_seq). Hmm, that would be a better, more thorough, fix I think. It might preclude some `since_seq` uses, say as a way to fast-forward an existing replication temporarily and since it changes the replication checkpoints IDs, it would be a larger compatibility issue. But if it makes the parameter behavior more consistent and predictable, I'd be for it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
