Add missing {DBI} methods

[vladimirobucina]

Based on dicussion and this comment: #47 (comment) I propose adding missing methods for {DBI} interface.

Mainly:

• dbReadTable
• dbCreateTable
• dbRemoveTable

To add to that, using Statement Execution, along with ARROW_STREAM option set on, can improve reading table speed and should be used for larger tables (probably we should not use INLINE at all)

@zacdav-db hope this is good enough of a description

[zacdav-db]

I'll look at this next week.

To add to that, using Statement Execution, along with ARROW_STREAM option set on, can improve reading table speed and should be used for larger tables (probably we should not use INLINE at all)

A lot of brickster already uses external links + arrow stream by default - if theres any particular part you want to discuss permalink it here please.

[zacdav-db]

I'll consider the following initially:

• dbRemoveTable()
• dbReadTable()
• dbCreateTable()

I'm interested in the arrow methods too, but want to wait until its not experimental. Users can use db_sql_query() with return_arrow until then.

Funnily enough I think db_create_table_from_data() is basically dbCreateTable() already.
dbReadTable() seems like its just a wrapper for dbGetQuery("SELECT * FROM <table>").

[zacdav-db]

@vladimirobucina can you please test #149 to see if that's behaving as you expect?

I've added the three methods.

[vladimirobucina]

@zacdav-db will do

[vladimirobucina]

@zacdav-db it looks to me like it's working well, I've tested it and no issues.