feat: Use IEnumerable in WriteApi to eliminate unnecessary memory all…

[birdalicious]

Proposed Changes

Change WriteApi to use IEnumerable for collection arguments instead of List.
Change WriteRecords(List<string> records) to WriteRecords(IEnumerable<string> records)

Calling ToList on a collection will incur a memory allocation as the data is copied to the new list. Making the argument IEnumerable means ToList doesn't need to be called and so no extra memory allocation.
Internally this will eliminate the unnecessary memory allocation in WriteRecords(string[] record) call to ToList, and client code also won't have to make the allocation if their data isn't already in a List## Checklist

Checklist

• CHANGELOG.md updated
• Rebased/mergeable
• A test has been added if appropriate
• dotnet test completes successfully
• Commit messages are conventional
• Sign CLA (if not already signed)

[birdalicious]

Sorry for the long delay in updating this PR. I've attempted to run the test suit but most the tests fail on
The request was canceled due to the configured HttpClient.Timeout of 10 seconds elapsing. or similar error messages. I've checked the master branch also does this for me. Is there something I'm missing, I've ran the influx-restart.sh script and have the containers running.

[bednar]

Can you check that the InfluxDB 2 is running on the port 9999 by: curl -i -X GET 'http://localhost:9999/health' ?

[birdalicious]

It is running and the server replies, all three are up and replying

[bednar]

Thanks for your PR 👍

Please satisfy following code formatting check: https://app.circleci.com/pipelines/github/influxdata/influxdb-client-csharp/3171/workflows/0946dbc5-63a0-43f2-b57f-d82a06bdda65/jobs/13158

[birdalicious]

@bednar fixed the code formatting

[rogerjak]

Will this ever be merged? Or is this project dead?

[bednar]

LGTM 🚀