Bug: `DELETE` gives strange `Cannot perform multiplication` error about surrealdb HOT 7 CLOSED

This falls out of my scope, but being aware of how many times I have tagged - assigning to myself. Going to bring up in next eng meet about how we spread and prioritise workload.

For the ticket: It would seem that there is processing on the document, but I don't understand how that would happen since deletes don't really process the document. @DelSkayn if you have any input that would be appreciated!

from surrealdb.

I am looking into this but I am currently getting a different error. I downloaded the surreal deal dataset and imported it successfully. I am however facing an issue the order relation. All orders are related via a order record id which consists of a country and seemingly a datetime. However the datetime is in the form of 2023-01-04T03:55:50.718612. This is an invalid datetime as it is missing a timezone. So the current error I am getting is that 2023-01-04T03:55:50.718612 can't be converted to a datetime.

from surrealdb.

Ive noticed errors during deletes because of futures/events that are getting fired that are now invalid based on the calculation of the deleted data.
Mine was a event that accessed $after.field, and that field was a future that became "broken" from the DELETE, so it was a underlying query not the actual DELETE that was erroring.

Is this related to those records that cause issues? (not got access to surreal to test currently)

from surrealdb.

So the current error I am getting is that 2023-01-04T03:55:50.718612 can't be converted to a datetime.

Interesting, once I upgraded to v1.1 I also get that error 🤔

from surrealdb.

The datetime error seems to be a different error: Previously there was some dubious behavior when parsing date-times. If only a part of the string matched a datetime it would still parse the datetime. so 2012-04-23blablabla would parse the 2012-04-23 part and ignore the blablabla. So with 2012-04-23T18:25:43.5110000 it fails to parse the full datetime since the time zone is missing and therefore would only parse 2012-04-23 . Now it errors instead if there is still some text left which causes the error I found.

Updating the dataset to have the correct datetimes makes the multiplication error return.

from surrealdb.

I believe this issue can be closed. The problem seemed to be inside one of the queries where it would try to multiply a number times a null because of a missing else branch so the error message was actually correct.

We still have a problem that the error message here is not that useful but that should probably be a different issue.

from surrealdb.

For those interested what the actual problem was, in the sureal-deal dataset there was the query:

DEFINE TABLE daily_sales SCHEMALESS AS 
    SELECT count() AS number_of_orders, 
        time::format(<datetime> order_date, '%Y-%m-%d') AS day, 
        math::sum((price * IF discount = NONE THEN 1 END) * quantity) AS sum_sales, 
        currency FROM order GROUP BY day, currency;

Which should have been:

DEFINE TABLE daily_sales SCHEMALESS AS 
    SELECT count() AS number_of_orders, 
        time::format(<datetime> order_date, '%Y-%m-%d') AS day, 
        math::sum((price * IF discount = NONE THEN 1 ELSE discount END) * quantity) AS sum_sales, 
        currency FROM order GROUP BY day, currency;

from surrealdb.