<div><div dir="auto">Liviu,</div><div dir="auto">In my opinion, way too much work for almost no value. I wouldn’t worry about it. I think the 8 digits of precision is ok for now really. Certainly meets my needs.  </div><div dir="auto"><br></div><div dir="auto">As a middle ground, instead of extending db_val_t, what if you used the cues from the DB driver to enforce the digits of precision while casting to a STRING. The problem with how you do it now is that you assume the precision which might actually be wrong. </div><div dir="auto"><br></div><div dir="auto">-Brett</div></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 11, 2020 at 10:40 AM Liviu Chircu <<a href="mailto:liviu@opensips.org">liviu@opensips.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On 10.03.2020 20:55, Brett Nemeroff wrote:<br>
> Can you inspect the DB type to derive a precision for the STRING <br>
> format? Then maybe default to 8 if you can't derive it?<br>
<br>
We probably could, but looking at db/db_val.h +75, you can see that the <br>
generic db_val_t type has no support for storing the precision.  So we'd <br>
have to:<br>
<br>
* extend this struct so it includes "precision digits" for floating <br>
point types.  Hopefully with 0 side-effects!<br>
* add some handy get/set macros for the above<br>
* for the MySQL driver: inspect the column properties (it has to be <br>
possible) and extract/store the decimal digits into the result<br>
* for other SQL drivers: feature not implemented for now?!<br>
* in avpops, make use of the db_val_t precision digits and finally use <br>
them to properly format the output data<br>
<br>
So this seems like too much of a hassle for a minor (if any) <br>
improvement, at least in my opinion.  There are other tasks far more <br>
important to be done for 3.1 instead of this little quirk.<br>
<br>
PS: Yesterday, I already pushed the 8-digits patch upstream anyway :)<br></blockquote></div></div>