On 2025-Nov-11, Chao Li wrote:
> I quickly went through this patch and got a big concern. From what I
> have learned, “executor" depends on “access”. To prove that, I quickly
> browse through a bunch of files under executor and access, which
> showed me that files under executor can include access
> On Nov 10, 2025, at 21:48, Mario González Troncoso
> wrote:
>
> Hey there,
>
> Based on the Alvaro's idea [1] about moving different instrumentation
> related C structures and enums into one single header file, I'm
> sending the following patches.
> That single file is named `executor/inst
Hey there,
Based on the Alvaro's idea [1] about moving different instrumentation
related C structures and enums into one single header file, I'm
sending the following patches.
That single file is named `executor/instrument_node.h`
Local tests and CI tests are passing
https://cirrus-ci.com/build/6
