crate / sql-99

SQL-99 Complete, Really
Other
22 stars 9 forks source link

Missing '(' in formula #74

Open jlb555555 opened 2 years ago

jlb555555 commented 2 years ago

Documentation feedback


Text: Example: Try passing the expression: 1.000001 + 1.999990) * 11000 to our calculator. Our proc gives the correct result: “33000.0000000”. The compilers we tested gave the wrong result: “32999.9901000”. (Remember that in SQL all the s in this expression are DECIMAL, not floating-point, s.)

Should read: Example: Try passing the expression: (1.000001 + 1.999990) * 11000 to our calculator. Our proc gives the correct result: “33000.0000000”. The compilers we tested gave the wrong result: “32999.9901000”. (Remember that in SQL all the s in this expression are DECIMAL, not floating-point, s.)

Entered SELECT (1.000001 + 1.999990) * 11000; in crash command line in Windows. Output: +-----------+ | 32999.901 | +-----------+ | 32999.901 | +-----------+

Should be “33000.0000000” ????

amotl commented 2 years ago

Dear John,

thank you for your feedback on this topic and apologies for the late reply. The content within this repository is a verbatim copy of the SQL-99 Complete, Really book. In order to reproduce its original content, we will probably not make any changes to it.

However, we could think about adding a custom errata section if that would fit your bill. Please submit a respective pull request if you think that would improve the situation.

Entered SELECT (1.000001 + 1.999990) * 11000; in crash command line in Windows.

This book reflects the SQL-99 standard, not the specific behavior of CrateDB. We are currently in the progress of improving the situation with the published SQL-99 book by giving it a more distinct look in order to avoid any confusion between CrateDB SQL and SQL-99. We will let you know as soon as this has been concluded.

On the other hand, I would like to encourage you to create an issue over at https://github.com/crate/crate if you think you discovered a bug within CrateDB.

With kind regards, Andreas.

jlb555555 commented 2 years ago

Hi Andreas, Thank you for your communication. I don't know what a pull request is or how to do it. In fact I would just create utter chaos if I try. Please proceed as you would see fit to deal with this. Many many thanks for your time.

On Fri, 4 Mar 2022 at 10:06, Andreas Motl @.***> wrote:

Dear John,

thanks for your feedback on this topic. The content within this repository is a verbatim copy of the SQL-99 Complete, Really https://openlibrary.org/books/OL8128443M/SQL-99_Complete_Really book. In order to reproduce its original content, we will probably not make any changes to it.

However, we could think about adding a custom errata section if that would fit your bill. Please submit a respective pull request if you think that would improve the situation.

Entered SELECT (1.000001 + 1.999990) * 11000; in crash command line in Windows.

This book reflects the SQL-99 standard, not the specific behavior of CrateDB. We are currently in the progress of improving the situation with the published SQL-99 book by giving it a more distinct look in order to avoid any confusion between CrateDB SQL and SQL-99. We will let you know as soon as this has been concluded.

With kind regards, Andreas.

— Reply to this email directly, view it on GitHub https://github.com/crate/sql-99/issues/74#issuecomment-1058929392, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE5VTQ7RJPDW7EKKMJVT55TU6G76PANCNFSM5L3GOVJQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

-- John Le Brasseur 073 157 5605