joao-r-reis commented on PR #1943: URL: https://github.com/apache/cassandra-gocql-driver/pull/1943#issuecomment-5037259986
> One of my major takeaways from this though, was the potential for exec intercepters to effectively replace `[Batch|Query]Observer`s with a little work, for the sake of keeping the API simple for users, rather than having two close/competing APIs > > * To be clear, I'm not talking about deleting `[Batch|Query]Observer` immedietly, I'm just suggesting we could somewhat deprecate them or otherwise put them on the back burner, especially as it's marked w/ an `// Experimental, this interface and use may change` comment. We could fairly easily just keep them and actually reimplement them under the hood as exec interceptors so we don't need to duplicate related code/mechanisms while keeping them entirely backwards compatable I think this is something we should keep in mind going forward but we can work on this new Interceptor feature without having to worry too much about the Observers right now. Conceptually they are two different things and they can exist independently of each other but in the future we might try to just reimplement observers so they are interceptors under the hood. Something to consider in the future but out of scope for this current PR -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
