Traces / analysis / tests
- Dark feels unsafe to users
- size of traces is out of control
- when moving around the editor, we don't stay in the same trace
- nothing works if you don't have traces
Storage of traces
Problem: the storage of traces is poor. Each trace is currently stored and then GCed - the performance of the GC is poor, despite multiple incremental fixes, and causes operational issues.
Solution: The trace feature should be redesigned with performance taken into account. The access patterns are:
- traces are created on each request. They are created in several parts and written to the DB each time. The inputs are written on input (they are stored with the path, module, and modifier instead of a tlid, to allow searching for 404s. It also means that 404s are created dynamically. The intent was for 404s to be automatically repopulated when changing the name of a canvas: this feature is alsp confusing to users)
- traces are updated via:
So
- store 404s separately
- new handler and function executions get new traces
- store trace data in S3/Cloud Storage
- add rate limiting so we only store a subset of traces (make configurable)
Traces don't work well in functions
Problem: when you create code in a function for the first time, the function does not have a trace, and the arguments are Incomplete
. This means everything is red and when you try to call functions, nothing works.
Why? The system is built around trace-driven development, but if you don't have a trace, nothing works.
Solution: Add a warning that you're using the default trace and it doesn't have values for the arguments. Fade the Play button and show a warning about this. Or at least give an error when you try to run something with an incomplete in it.
Solution: Perhaps default traces should use default values for known types. However, that will result in real values being put in the DB, which the user will then have to dig out.
Solution: perhaps a dry-run of some sort might be an idea in this situation?