apache / apache/arrow

[C++][FlightSQL][ODBC]: Use `DCHECK` macro instead of `assert`

Open
#47,990 0 comments 0 reactions 0 assignees View on GitHub
Component: C++ Component: FlightRPC Type: enhancement
Dominant language
C++
Stars
17.1k
Forks
4.3k
Avg merge
3d 13h
Merged PRs (30d)
88

Description

### Describe the enhancement requested

Use `DCHECK` macro instead of `assert` in `cpp\src\arrow\flight\sql\odbc\odbc_impl` folder

Parent issue: https://github.com/apache/arrow/issues/30622

### Component(s)

C++, FlightRPC

Contributor guide

Open the contributing guide

Research direction

Start by searching cpp/src/arrow/flight/sql/odbc/odbc_impl for assert usages and inspect nearby C++ assertion conventions. The work is done when the applicable assertions in that folder use DCHECK instead, with the relevant C++ and FlightSQL/ODBC tests or build checks passing.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.