tiberius icon indicating copy to clipboard operation
tiberius copied to clipboard

Allow Row to get ColumnData

Open LazyDope opened this issue 1 year ago • 4 comments

When working on data of an unknown type, it's very difficult to get a specific index without explicitly giving the type that we are going to receive. Being able to handle the ColumnData instead of needing to provide a concrete type that implements FromSql allows this to be handled more conveniently.

LazyDope avatar Jul 17 '23 15:07 LazyDope

I second this! Need to query a single cell, but I don't know its result type in advance. ColumnData would have all the required information, but the stored ColumnData of a Row is currently not exposed. This PR or any similar getter for Row would fix that.

CC @miguelff @pimeys @steffengy @Weakky

ColinFinck avatar Aug 01 '23 15:08 ColinFinck

An acceptable alternative would be implementing FromSql for ColumnData that returns a clone of the ColumnData, but most of the implementation was already present for this, it was just a matter of separating it into its own function.

LazyDope avatar Aug 01 '23 16:08 LazyDope

Does the changes in #303 address this case? Granted, you will have to iterate to the column you want, but unless you have wide result sets or serious performance requirements, that might be acceptable?

bittrance avatar Aug 30 '23 08:08 bittrance

No, the borrow checker was not really the problem for me, it is much more straightforward for me to be able to get the data with the column name rather than iterating to the correct column, which is why I make this change in the first place.

LazyDope avatar Aug 30 '23 16:08 LazyDope