feat: Add dbConnect(array = "none"), pass array = "matrix" to enable conversion of array columns

[joakimlinde]

No description provided.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 93.87755% with 3 lines in your changes missing coverage. Please review.

Project coverage is 51.21%. Comparing base (5c88c6f) to head (98a4bd9).
Report is 1495 commits behind head on main.

Files with missing lines	Patch %	Lines
src/transform.cpp	92.85%	2 Missing ⚠️
src/convert.cpp	85.71%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1125       +/-   ##
===========================================
- Coverage   63.61%   51.21%   -12.40%     
===========================================
  Files         123      146       +23     
  Lines        5266     9920     +4654     
===========================================
+ Hits         3350     5081     +1731     
- Misses       1916     4839     +2923     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[joakimlinde]

Seems like the 'arrow' package is not available on R 4.1: Package suggested but not available for checking: ‘arrow’

[krlmlr]

Thanks. The R 4.1 check failures are on me.

Do you think it's useful to split this into two PRs, and start with a refactoring PR that replaces integer64 with convert_opts everywhere? This would be easier to reason about later. I'm not sure if it's really worth investing that time now -- we can always do it after the fact if needed.

[krlmlr]

I don't think bigint belongs here, this should be deprecated at some point. Let's not add "array" here. Should have been clearer about that in the code, add a comment to that effect now?

[krlmlr]

I'm mistaken, this is where array = "none" belongs. For some reason I thought this is the duckdb() function.

[joakimlinde]

The PR is fine as it is. Don't worry about it.

[krlmlr]

Thanks!