-
Notifications
You must be signed in to change notification settings - Fork 277
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lazy queries inside WASM #3929
Comments
This is not the reason to implement lazy queries in wasm. If query is executed from Executor the result should be cached on the host (i.e. not returned to the Executor) during validation because there is no point in dragging the result through Executor again |
not to mention that it's a security issue to allow references to flow from host into smartcontract. A malicious smartcontract could execute several such queries until it induces OOM on the host |
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Daniil Polyakov <[email protected]>
We support lazy (kind-of) queries for basic clients, but it's not supported for WASM.
This creates some problems. For example because of that we cannot move our queries execution into Executor causing stragne divergence: transctions and instructions are executed, but queries are not.
Need to implement lazy evaluation for WASM as well
The text was updated successfully, but these errors were encountered: