Remove use of deprecated record_batches_to_json_rows function from arrow-json

[hiltontj]

We are using a function from arrow-json crate that is deprecated: record_batches_to_json_rows

It is used in the following places:

1. influxdb/influxdb3_server/src/http.rs

   Line 810 in a5eba2f

   &arrow_json::writer::record_batches_to_json_rows(batches.as_slice())?,

2. influxdb/influxdb3_server/src/http/v1.rs

   Line 342 in a5eba2f

   let json_rows = record_batches_to_json_rows(&[&batch])

The deprecation warning indicates that we should be using Writer. Doing so in the latter case should be easier since we are serializing directly to the bytes outputted in the API response, however, the former may be a bit trickier, as we are modifying the serialized JSON values before serializing to bytes in the response.

[JeanArhancet]

I am interested in taking on this task. Is it possible to assign this ticket to me?

[hiltontj]

Hi @JeanArhancet - that would be awesome, thank you! This issue is not in our immediate backlog, so if you would like to tackle it by opening a PR, you are welcome to do that.

My recommendation is to start by solving (1.) in the issue description, and opening a PR for only that. Since that is using arrow-json's writer for its intended purpose: serializing and writing JSON. Specifically, I would look at arrow-json's ArrayWriter, as that serializes to a JSON array, as opposed to JSON Lines. Separately, we do intend to support JSON Lines (see #24654).

As for (2.), the use of record_batches_to_json_rows is a bit of a hack there, so the solution is not to use the arrow-json writer, but will be something different.

Please keep in mind that you will need to read and sign our CLA in order for contributions to be accepted. You can find that here: https://www.influxdata.com/legal/cla/

[JeanArhancet]

Thanks for your feedback, @hiltontj.

I've opened a PR for the first part: #25046