ydb-go-sdk icon indicating copy to clipboard operation
ydb-go-sdk copied to clipboard

feat: standardization/unification for Query

Open flymedllva opened this issue 1 year ago • 5 comments

Feature Request

I want to write a wrapper package that uses Execute/ReadRow/ReadResultSet from Session and TxActor, encountered that the signature of the methods became very different

  1. options package is imported from internal, so I cannot describe a partial interface consisting of 1 method.
  2. for Session & TxActoroptions have become different – ExecuteOption & TxExecuteOption, which prevents you from using 1 interface for them
  3. I don't like the fact that the Execute call in Session gives Transaction, because of this method signature is different from TxActor, which does not allow to use 1 interface, as I used to do with database/sql. That said, I understand the idea of needing to return Transaction, but maybe in TxActor, it should also be returned?
Screenshot 2024-06-09 at 17 55 25

flymedllva avatar Jun 09 '24 15:06 flymedllva