Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion .github/workflows/cpp_extra.yml
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,7 @@ jobs:
needs.check-labels.outputs.force == 'true' ||
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra') ||
contains(fromJSON(needs.check-labels.outputs.ci-extra-labels || '[]'), 'CI: Extra: C++')
timeout-minutes: 75
timeout-minutes: 120
strategy:
fail-fast: false
matrix:
Expand All @@ -429,6 +429,7 @@ jobs:
ARROW_DEPENDENCY_SOURCE: BUNDLED
ARROW_DEPENDENCY_USE_SHARED: OFF
ARROW_FLIGHT_SQL_ODBC: ON
ARROW_FLIGHT_SQL_ODBC_INSTALLER: ON
ARROW_HOME: /tmp/local
ARROW_MIMALLOC: OFF
steps:
Expand Down Expand Up @@ -504,6 +505,19 @@ jobs:
--allow libresolv \
--allow libz \
"$(pwd)/build/cpp/${{ matrix.build-type }}/libarrow_flight_sql_odbc.dylib"
- name: Generate macOS Installer
if: matrix.build-type == 'release'
shell: bash
run: |
cd $(pwd)/build/cpp
cpack
- name: Upload ODBC PKG to the job
if: matrix.build-type == 'release'
uses: actions/upload-artifact@v7
with:
name: flight-sql-odbc-pkg-installer-${{ matrix.architecture }}
path: build/cpp/ArrowFlightSqlOdbcODBC-*.pkg
if-no-files-found: error
- name: Register Flight SQL ODBC Driver
run: |
sudo cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc.sh $(pwd)/build/cpp/${{ matrix.build-type }}/libarrow_flight_sql_odbc.dylib
Expand Down
2 changes: 2 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,9 @@ repos:
?^cpp/build-support/update-thrift\.sh$|
?^cpp/examples/minimal_build/run\.sh$|
?^cpp/examples/tutorial_examples/run\.sh$|
?^cpp/src/arrow/flight/sql/odbc/install/mac/postinstall$|
?^cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc\.sh$|
?^cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc_ini\.sh$|
?^dev/release/05-binary-upload\.sh$|
?^dev/release/08-binary-verify\.sh$|
?^dev/release/binary-recover\.sh$|
Expand Down
75 changes: 73 additions & 2 deletions cpp/src/arrow/flight/sql/odbc/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,6 @@ if(ARROW_FLIGHT_SQL_ODBC_INSTALLER)
set(CPACK_PACKAGE_DESCRIPTION_SUMMARY "Apache Arrow Flight SQL ODBC Driver")
set(CPACK_PACKAGE_CONTACT "dev@arrow.apache.org")

# GH-47876 TODO: set up `arrow_flight_sql_odbc` component for macOS Installer
# GH-47877 TODO: set up `arrow_flight_sql_odbc` component for Linux Installer
if(WIN32)
# Install ODBC and its Arrow dependencies
Expand All @@ -159,6 +158,66 @@ if(ARROW_FLIGHT_SQL_ODBC_INSTALLER)

set(CPACK_WIX_UI_BANNER
"${CMAKE_CURRENT_SOURCE_DIR}/install/windows/arrow-wix-banner.bmp")
else()
if(APPLE)
set(CPACK_PACKAGE_FILE_NAME
"ArrowFlightSqlOdbcODBC-${CPACK_PACKAGE_VERSION_MAJOR}.${ODBC_PACKAGE_VERSION_MINOR}.${ODBC_PACKAGE_VERSION_PATCH}"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the OdbcODBC part intentional?

)
set(CPACK_PACKAGE_INSTALL_DIRECTORY "${CPACK_PACKAGE_NAME}")

set(CPACK_SET_DESTDIR ON)
set(CPACK_INSTALL_PREFIX "/Library/ODBC")
# Register ODBC after install
set(CPACK_POSTFLIGHT_ARROW_FLIGHT_SQL_ODBC_SCRIPT
"${CMAKE_CURRENT_SOURCE_DIR}/install/mac/postinstall")
set(CPACK_RESOURCE_FILE_README "${CMAKE_CURRENT_SOURCE_DIR}/install/mac/README.txt")
set(CPACK_RESOURCE_FILE_WELCOME
"${CMAKE_CURRENT_SOURCE_DIR}/install/mac/Welcome.txt")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use lower case (welcome.txt)?


set(ODBC_INSTALL_DIR "arrow-odbc/lib")
set(DOC_INSTALL_DIR "arrow-odbc/doc")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you use ODBC_ prefix?

else()
Comment on lines +168 to +179
Copy link

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The linked issue description calls out installing the driver under /Library/Apache/ArrowFlightSQLODBC/lib/..., but this packaging installs under /Library/ODBC/arrow-odbc/... (via CPACK_INSTALL_PREFIX and ODBC_INSTALL_DIR). Please confirm the intended install root and align with the issue (or update the issue/PR description) to avoid shipping an installer that doesn't meet the documented path expectation.

Copilot uses AI. Check for mistakes.
# Linux
# GH-49595: TODO implement DEB installer
# GH-47977: TODO implement RPM installer
message(STATUS "ODBC_PACKAGE_FORMAT DEB not implemented, see GH-49595")
message(STATUS "ODBC_PACKAGE_FORMAT RPM not implemented, see GH-47977")
Comment on lines +181 to +184
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to use CPack for deb and RPM?
I want to use https://github.com/apache/arrow/tree/main/dev/tasks/linux-packages for them.

endif()

# Install ODBC
install(TARGETS arrow_flight_sql_odbc_shared
DESTINATION "${ODBC_INSTALL_DIR}"
COMPONENT arrow_flight_sql_odbc)

# Install temporary driver registration scripts, they will be removed after driver registration is complete
install(FILES "${CMAKE_CURRENT_SOURCE_DIR}/install/unix/install_odbc.sh"
DESTINATION "${ODBC_INSTALL_DIR}"
COMPONENT arrow_flight_sql_odbc
PERMISSIONS OWNER_EXECUTE
OWNER_WRITE
OWNER_READ
GROUP_EXECUTE
GROUP_READ
WORLD_EXECUTE
WORLD_READ)
install(FILES "${CMAKE_CURRENT_SOURCE_DIR}/install/unix/install_odbc_ini.sh"
DESTINATION "${ODBC_INSTALL_DIR}"
COMPONENT arrow_flight_sql_odbc
PERMISSIONS OWNER_EXECUTE
OWNER_WRITE
OWNER_READ
GROUP_EXECUTE
GROUP_READ
WORLD_EXECUTE
WORLD_READ)

# Install documentation files
install(FILES "${CMAKE_CURRENT_SOURCE_DIR}/../../../../../../LICENSE.txt"
DESTINATION "${DOC_INSTALL_DIR}"
COMPONENT Docs)
Comment on lines +215 to +217
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LICENSE.txt (and NOTICE.txt) are already installed to ARROW_DOC_DIR. Do we need to install them separately for ODBC?

install(FILES "${CMAKE_CURRENT_SOURCE_DIR}/Connection-Options.md"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use lower case (connection-options.md)?

DESTINATION "${DOC_INSTALL_DIR}"
COMPONENT Docs)
Comment on lines +218 to +220
Copy link

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Connection-Options.md is installed into the package (Docs component) but currently only contains a TODO placeholder. Since this is user-facing documentation shipped with the installer, it should either be populated with the supported keys/options or omitted from installation until it has real content.

Suggested change
install(FILES "${CMAKE_CURRENT_SOURCE_DIR}/Connection-Options.md"
DESTINATION "${DOC_INSTALL_DIR}"
COMPONENT Docs)

Copilot uses AI. Check for mistakes.
endif()

get_cmake_property(CPACK_COMPONENTS_ALL COMPONENTS)
Expand All @@ -173,8 +232,13 @@ if(ARROW_FLIGHT_SQL_ODBC_INSTALLER)
# Upgrade GUID is required to be unchanged for ODBC installer to upgrade
set(CPACK_WIX_UPGRADE_GUID "DBF27A18-F8BF-423F-9E3A-957414D52C4B")
set(CPACK_WIX_PRODUCT_GUID "279D087B-93B5-4DC3-BA69-BCF485022A26")
else()
# macOS and Linux
list(APPEND CPACK_COMPONENTS_ALL Docs)
if(APPLE)
set(CPACK_GENERATOR "productbuild")
endif()
endif()
# GH-47876 TODO: create macOS Installer using cpack
# GH-47877 TODO: create Linux Installer using cpack

# Load CPack after all CPACK* variables are set
Expand All @@ -183,4 +247,11 @@ if(ARROW_FLIGHT_SQL_ODBC_INSTALLER)
DISPLAY_NAME "ODBC library"
DESCRIPTION "Apache Arrow Flight SQL ODBC library bin, required to install"
REQUIRED)
if(UNIX)
# On macOS and Linux, provide connection string documentation since users need to manually enter DSN keys.
cpack_add_component(Docs
DISPLAY_NAME "Documentation"
DESCRIPTION "Documentation for Apache Arrow Flight SQL ODBC Driver"
)
endif()
endif()
20 changes: 20 additions & 0 deletions cpp/src/arrow/flight/sql/odbc/Connection-Options.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
<!---
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->

GH-49723 TODO: enter ODBC connection options for unix DSN
9 changes: 9 additions & 0 deletions cpp/src/arrow/flight/sql/odbc/install/mac/README.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
Files are available in '/Library/ODBC/arrow-odbc' after installation.

To setup a connection, you can use DSN to store your data source connection information.
1. Open 'iODBC Data Source Administrator'.
2. To create a user DSN, go to 'User DSN' tab and click 'Add'.
3. Select 'Apache Arrow Flight SQL ODBC Driver' and click 'Finish'.
4. Enter DSN name and connection string values.
For the list of all supported options, check '/Library/ODBC/arrow-odbc/doc/Connection-Options.md'.
5. Click 'Ok' to save the DSN.
Comment on lines +3 to +9
Copy link

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

User-facing text fixes: change "To setup a connection" to "To set up a connection" (grammar), change "Ok" to "OK", and remove the trailing whitespace at the end of line 7. Since this README is used as an installer resource, it should be polished.

Suggested change
To setup a connection, you can use DSN to store your data source connection information.
1. Open 'iODBC Data Source Administrator'.
2. To create a user DSN, go to 'User DSN' tab and click 'Add'.
3. Select 'Apache Arrow Flight SQL ODBC Driver' and click 'Finish'.
4. Enter DSN name and connection string values.
For the list of all supported options, check '/Library/ODBC/arrow-odbc/doc/Connection-Options.md'.
5. Click 'Ok' to save the DSN.
To set up a connection, you can use DSN to store your data source connection information.
1. Open 'iODBC Data Source Administrator'.
2. To create a user DSN, go to 'User DSN' tab and click 'Add'.
3. Select 'Apache Arrow Flight SQL ODBC Driver' and click 'Finish'.
4. Enter DSN name and connection string values.
For the list of all supported options, check '/Library/ODBC/arrow-odbc/doc/Connection-Options.md'.
5. Click 'OK' to save the DSN.

Copilot uses AI. Check for mistakes.
1 change: 1 addition & 0 deletions cpp/src/arrow/flight/sql/odbc/install/mac/Welcome.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Apache Arrow Flight SQL ODBC Driver is a read-only ODBC driver for connecting to data sources that support Arrow Flight SQL.
30 changes: 30 additions & 0 deletions cpp/src/arrow/flight/sql/odbc/install/mac/postinstall
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
#!/bin/bash
#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.

# Use temporary driver registration script to register ODBC driver in system DSN
odbc_install_script="/Library/ODBC/arrow-odbc/lib/install_odbc.sh"
"$odbc_install_script" /Library/ODBC/arrow-odbc/lib/libarrow_flight_sql_odbc.dylib

# Use temporary DSN registration script to register sample system DSN
dsn_install_script="/Library/ODBC/arrow-odbc/lib/install_odbc_ini.sh"
"$dsn_install_script" /Library/ODBC/odbc.ini

# clean temporary script
rm -f "$odbc_install_script"
rm -f "$dsn_install_script"
Comment on lines +20 to +30
Copy link

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The postinstall script doesn't enable strict error handling (e.g., set -euo pipefail). As written, if either registration script fails, the postinstall can still exit successfully because the final rm -f commands will return 0, leaving the driver/DSN unregistered while the installer reports success. Please fail fast on any error and consider emitting a clear error message if registration fails.

Copilot uses AI. Check for mistakes.
6 changes: 3 additions & 3 deletions cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
# specific language governing permissions and limitations
# under the License.

# Used by macOS ODBC installer script `install_odbc_ini.sh` and macOS ODBC testing
# Used by arrow/cpp/src/arrow/flight/sql/odbc/install/mac/postinstall

set -euo pipefail

Expand Down Expand Up @@ -46,8 +46,8 @@ case "$(uname)" in
;;
*)
# macOS
USER_ODBCINST_FILE="$HOME/Library/ODBC/odbcinst.ini"
mkdir -p "$HOME"/Library/ODBC
USER_ODBCINST_FILE="/Library/ODBC/odbcinst.ini"
mkdir -p /Library/ODBC
Comment on lines 48 to +50
Copy link

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On macOS this script now writes to the system-level /Library/ODBC/odbcinst.ini, but the variable is still named USER_ODBCINST_FILE. Renaming it to reflect that it's a system-wide file would reduce confusion for future maintenance (especially since the script requires sudo).

Copilot uses AI. Check for mistakes.
;;
esac

Expand Down
72 changes: 72 additions & 0 deletions cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc_ini.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,72 @@
#!/bin/bash
#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.

SYSTEM_ODBC_FILE="$1"

if [[ -z "$SYSTEM_ODBC_FILE" ]]; then
echo "error: path to system ODBC DSN is not specified. Call format: install_odbc_ini abs_path_to_odbc_dsn_ini"
exit 1
fi

DRIVER_NAME="Apache Arrow Flight SQL ODBC Driver"
DSN_NAME="Apache Arrow Flight SQL ODBC DSN"

touch "$SYSTEM_ODBC_FILE"

Comment on lines +20 to +31
Copy link

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script doesn't use strict mode (set -euo pipefail) like install_odbc.sh does. Without it, failures from commands like touch, awk, or mv may go unnoticed depending on where they occur, which is risky for an installer-time system configuration update. Please add strict mode and ensure any failures propagate as a non-zero exit code.

Copilot uses AI. Check for mistakes.
if grep -q "^\[$DSN_NAME\]" "$SYSTEM_ODBC_FILE"; then
echo "DSN [$DSN_NAME] already exists in $SYSTEM_ODBC_FILE"
else
echo "Adding [$DSN_NAME] to $SYSTEM_ODBC_FILE..."
cat >> "$SYSTEM_ODBC_FILE" <<EOF

[$DSN_NAME]
Description = An ODBC Driver DSN for Apache Arrow Flight SQL
Driver = Apache Arrow Flight SQL ODBC Driver
Host =
Port =
UID =
PWD =
EOF
fi

# Check if [ODBC Data Sources] section exists
if grep -q '^\[ODBC Data Sources\]' "$SYSTEM_ODBC_FILE"; then
# Section exists: check if DSN entry exists
if ! grep -q "^${DSN_NAME}=" "$SYSTEM_ODBC_FILE"; then
# Add DSN entry under [ODBC Data Sources] section

# Use awk to insert the line immediately after [ODBC Data Sources]
awk -v dsn="$DSN_NAME" -v driver="$DRIVER_NAME" '
$0 ~ /^\[ODBC Data Sources\]/ && !inserted {
print
print dsn "=" driver
inserted=1
next
}
{ print }
' "$SYSTEM_ODBC_FILE" > "${SYSTEM_ODBC_FILE}.tmp" && mv "${SYSTEM_ODBC_FILE}.tmp" "$SYSTEM_ODBC_FILE"
fi
Comment on lines +48 to +64
Copy link

Copilot AI Apr 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The DSN existence check under [ODBC Data Sources] only matches entries formatted exactly as Apache Arrow Flight SQL ODBC DSN=.... In many odbc.ini files the = is surrounded by whitespace (e.g., DSN_NAME = Driver), so this can fail to detect an existing entry and insert a duplicate. Consider matching optional whitespace around = (and/or parsing the section more robustly) before inserting.

Copilot uses AI. Check for mistakes.
else
# Section doesn't exist, append section and DSN entry at end
{
echo ""
echo "[ODBC Data Sources]"
echo "${DSN_NAME}=${DRIVER_NAME}"
} >> "$SYSTEM_ODBC_FILE"
fi
2 changes: 2 additions & 0 deletions dev/release/rat_exclude_files.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ ci/vcpkg/*.patch
CHANGELOG.md
cpp/CHANGELOG_PARQUET.md
cpp/src/arrow/c/dlpack_abi.h
cpp/src/arrow/flight/sql/odbc/install/mac/README.txt
cpp/src/arrow/flight/sql/odbc/install/mac/Welcome.txt
cpp/src/arrow/io/mman.h
cpp/src/arrow/util/random.h
cpp/src/arrow/status.cc
Expand Down
Loading