Corentin Leruth

Results 40 comments of Corentin Leruth

@ejgallego it doesn't seem I've the right to change the milestone. Do you mind doing it?

Thanks for the fix @dra27, I confirm that it resolves the build failure on `omd` 😁

Hey, what's the status of this PR? Although I don't have much experience in `C`, please let me know if I can do anything to help move this PR forward

Thanks for the review, I've applied the same changes to the `cohttp-lwt` lib

Today I used ```ocaml type application = { type_ : Aa_fe.Application.Type.t; [@json.name "type"] } ``` And was surprised it didn't work until I realized that I should use `json.key` instead....

Thanks for considering it. Also please let me know if you need help to rebase the branch on top of maser

I have rebased this PR on top of master (+ fixed conflicts) as agreed with @cyberhuman in an offline discussion

> the query fails because clickhouse requires all map values to be of the same type, and in this case we have (String, String, Map). I'm hitting the same issue...

Thanks for your input. It doesn't beak the UI but `attributes` aren't shown. I believe because it expects an `array` while it is a `string` ![Image](https://github.com/user-attachments/assets/f92f0d0d-6f2b-47b8-af8d-de7100766d59)

I found a solution for `events`, see https://github.com/grafana/clickhouse-datasource/discussions/840#discussioncomment-10799129. I guess we can apply the same solution for `Links.Attributes` ?