When providing a connection directly to the IRISVector() constructor, rather than the URL to create one, we're not verifying if the database we're connecting to supports the native vector datatype, as that check is only performed in connect().
The change in this PR is trivially fixing this inside IRISVector() because I'm not familiar with other uses of the connection, but either moving the check to a separate on_connect() method or changing the native_vectors property to dynamically check the connection rather than copy that state may be preferable long term.
When providing a connection directly to the
IRISVector()
constructor, rather than the URL to create one, we're not verifying if the database we're connecting to supports the native vector datatype, as that check is only performed inconnect()
.The change in this PR is trivially fixing this inside
IRISVector()
because I'm not familiar with other uses of the connection, but either moving the check to a separateon_connect()
method or changing thenative_vectors
property to dynamically check the connection rather than copy that state may be preferable long term.