You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
let aggregated = aggregated.as_ref().map(Rc::clone);
evaluate(storage, context, aggregated, expr)
};
enumNullable<T>{
Value(T),
Null,
}
When Nullable::Null comes, it always returns Ok(Evaluated::from(Value::Null)).
However, that result is also kind of success Ok. It cannot be handled by try operator ?.
It would be good to use macro until rfcs/3058-try-trait-v2.html comes to Rust stable.
This issue is to simplify
Nullable
enum control flow using declarative macro.gluesql/src/executor/evaluate/mod.rs
Lines 175 to 191 in de73106
When
Nullable::Null
comes, it always returnsOk(Evaluated::from(Value::Null))
.However, that result is also kind of success
Ok
. It cannot be handled by try operator?
.It would be good to use macro until rfcs/3058-try-trait-v2.html comes to Rust stable.
e.g.
gluesql/src/executor/evaluate/mod.rs
Lines 225 to 229 in de73106
We can change the above like this,
It is expected for us not to handle
Nullable
enum explicitly.The text was updated successfully, but these errors were encountered: