Ramnivas Laddad
Ramnivas Laddad
Given that we now have a plugin system (with its own set of issues for plugins outside of our tree) and (even if we overcome those issues), has potentially longer...
Consider the following exo file. ```exo @deno("core.ts") module Core { query getString(): String } @deno("intercept.ts") module Intercept { @around("query getString") interceptor bypassString() } ``` And the following intercept.ts: ```ts export...
Consider a timing interceptor, which returns the data from `operation.proceed()`. The proceed may have returned json output in the string form from the database. Yet, due to `DenoExecutor`s api, which...
Currently, `UseRequestContext` has a `pub transaction_holder: Arc,`. We need to clean this up to let each plugin keep some opaque data into `UseRequestContext` and in turn let the database plugin...
With plugins in place, it is weird to have, for example, core packages depend on a particular subsystem. See `create_database_system_from_str` as well as dev-dependencies to reexamine if we can avoid...
See `SystemLoader`. Currently, we build the schema schema during load time. We should shift this responsibility to build time. This might make loaders a bit faster, but more importantly, it...
Currently, we validate arguments to check that mandatory arguments are present and no stray arguments exists. For object arguments, we perform this checks recursively. But we do not check for...
Currently, we use those to build schema. However, those types aren't quite sufficient for validation (where we need to differentiate between an int and a float).
For a few integration tests, we depend on https://jsonplaceholder.typicode.com/. This can be sometimes flaky. Instead, we should create our own REST service and test against it. This will come handy...
When we migrated from "anyhow" to thiserror, we introduced `Generic` variant in the `#[error]` types. We should replace it with proper variants that will help in reporting and debugging.