issues: 1212823665

This data as json

id node_id number title user state locked assignee milestone comments created_at updated_at closed_at author_association pull_request body repo type active_lock_reason performed_via_github_app reactions draft state_reason
1212823665 I_kwDOBm6k_c5ISjhx 1715 Refactor TableView to use asyncinject 9599 closed 0     13 2022-04-22T21:43:39Z 2022-12-01T21:15:18Z 2022-04-28T22:26:56Z OWNER  

I've been working on a dependency injection mechanism in a separate library:

I think it's ready to try out with Datasette to see if it's a pattern that will work here.

I'm going to attempt to refactor TableView to use it. There are two overall goals here:

  • Use asyncinject to add parallel execution of some aspects of the table page - most notably I want to be able to execute the count(*) query, the select ... query, the various faceting queries and the facet suggestion queries in parallel - and measure if doing so is good for performance.
  • Use it to execute different output formats (possibly with some changes to the existing register_output_renderer() plugin hook). I want CSV and JSON to use the same mechanism that plugins use.

Stretch goal is to get this working with streaming data too, see:

  • 1101

 107914493 issue     
{
    "url": "https://api.github.com/repos/simonw/datasette/issues/1715/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   completed

Links from other tables