apache / apache/arrow-adbc

Go: Implement adbc.GetSetOptions on the Go ADBC drivermgr.cnxn connection type

Open Beginner friendly
#3,143 0 comments 0 reactions 0 assignees View on GitHub
Type: enhancement
Dominant language
C#
Stars
627
Forks
217
Avg merge
22h 22m
Merged PRs (30d)
64

Description

Problem:
Calling cnxn.(adbc.GetSetOptions).SetOption(adbc.OptionKeyCurrentCatalog, "random_data") panics because the drivermgr.cnxn type doesn’t implement the full adbc.GetSetOptions interface (missing GetOption).

Solution:
Add the missing GetOption method (and any other required stubs) to the Go driver’s cnxn type so it satisfies adbc.GetSetOptions, allowing safe use of SetOption and GetOption without panics.

Contributor guide

Open the contributing guide

Research direction

Inspect the Go drivermgr.cnxn connection type and its existing option methods. Confirm the requirements of adbc.GetSetOptions, then implement the missing GetOption and any other required stubs. Done means cnxn satisfies the interface and SetOption and GetOption can be called without the reported panic.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
databases
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.