Skip to content

Conversation

czentgr
Copy link
Contributor

@czentgr czentgr commented Aug 18, 2025

This PR adds unit tests to be run for the arrow flight connector. Previously, only the end-to-end tests were run in the separate CI job. For regular CI build & test arrow flight is not enabled.

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== NO RELEASE NOTE ==

@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Aug 18, 2025
@czentgr czentgr force-pushed the cz_add_arrow_flight_unit_ci branch from c9b3b72 to 0f6a117 Compare August 19, 2025 14:34
@czentgr czentgr marked this pull request as ready for review August 19, 2025 16:49
@czentgr czentgr requested review from unidevel and a team as code owners August 19, 2025 16:49
@prestodb-ci prestodb-ci requested review from a team, jkhaliqi and pramodsatya and removed request for a team August 19, 2025 16:49
This PR adds unit tests to be run for the arrow flight connector.
Previously, only the end-to-end tests were run in the separate CI job.
For regular CI build & test arrow flight is not enabled.
@czentgr czentgr force-pushed the cz_add_arrow_flight_unit_ci branch from 0f6a117 to f48e418 Compare August 20, 2025 03:29
Copy link
Contributor

@unidevel unidevel left a comment

Choose a reason for hiding this comment

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

LGTM

@czentgr czentgr merged commit 2bc42ae into prestodb:master Aug 21, 2025
69 of 70 checks passed
@czentgr czentgr deleted the cz_add_arrow_flight_unit_ci branch August 21, 2025 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
from:IBM PR from IBM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants