16728097825

Committed 04 Aug 2025 04:00PM UTC coverage: 48.355% (-35.1%) from 83.429%

Build # 16728097825

Build Type

Pull #4108

github

Committed by

web-flow

Commit Message

Merge 1b2d27fd8 into 649ba9576

Pull Request Pull Request #4108: perf[vortex-array]: use all_valid instead of `invalid_count() == 0`

Run Details

1 of 1 new or added line in 1 file covered. (100.0%)

11596 existing lines in 378 files now uncovered.

18635 of 38538 relevant lines covered (48.35%)

151786.4 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/vortex-duckdb/src/duckdb/table_function/partition.rs

// SPDX-License-Identifier: Apache-2.0
// SPDX-FileCopyrightText: Copyright the Vortex contributors

use std::ffi::c_void;

use cpp::duckdb_vx_error;
use vortex::error::VortexExpect;

use crate::cpp;
use crate::cpp::idx_t;
use crate::duckdb::TableFunction;

/// Native callback for the cardinality estimate of a table function.
pub(crate) unsafe extern "C-unwind" fn get_partition_data_callback<T: TableFunction>(
    bind_data: *const c_void,
    global_init_data: *mut c_void,
    local_init_data: *mut c_void,
    error_out: *mut duckdb_vx_error,
) -> idx_t {
    let bind_data =
        unsafe { bind_data.cast::<T::BindData>().as_ref() }.vortex_expect("bind_data null pointer");
    let global_init_data = unsafe { global_init_data.cast::<T::GlobalState>().as_mut() }
        .vortex_expect("global_init_data null pointer");
    let local_init_data = unsafe { local_init_data.cast::<T::LocalState>().as_mut() }
        .vortex_expect("local_init_data null pointer");

    match T::partition_data(bind_data, global_init_data, local_init_data) {
        Ok(batch_id) => batch_id,
        Err(e) => {
            // Set the error in the error output.
            let msg = e.to_string();
            unsafe { error_out.write(cpp::duckdb_vx_error_create(msg.as_ptr().cast(), msg.len())) };
            0
        }
    }
}

1	// SPDX-License-Identifier: Apache-2.0
2	// SPDX-FileCopyrightText: Copyright the Vortex contributors
3
4	use std::ffi::c_void;
5
6	use cpp::duckdb_vx_error;
7	use vortex::error::VortexExpect;
8
9	use crate::cpp;
10	use crate::cpp::idx_t;
11	use crate::duckdb::TableFunction;
12
13	/// Native callback for the cardinality estimate of a table function.
UNCOV 14	pub(crate) unsafe extern "C-unwind" fn get_partition_data_callback<T: TableFunction>(	×
UNCOV 15	bind_data: *const c_void,	×
UNCOV 16	global_init_data: *mut c_void,	×
UNCOV 17	local_init_data: *mut c_void,	×
UNCOV 18	error_out: *mut duckdb_vx_error,	×
UNCOV 19	) -> idx_t {	×
UNCOV 20	let bind_data =	×
UNCOV 21	unsafe { bind_data.cast::<T::BindData>().as_ref() }.vortex_expect("bind_data null pointer");	×
UNCOV 22	let global_init_data = unsafe { global_init_data.cast::<T::GlobalState>().as_mut() }	×
UNCOV 23	.vortex_expect("global_init_data null pointer");	×
UNCOV 24	let local_init_data = unsafe { local_init_data.cast::<T::LocalState>().as_mut() }	×
UNCOV 25	.vortex_expect("local_init_data null pointer");	×
26
UNCOV 27	match T::partition_data(bind_data, global_init_data, local_init_data) {	×
UNCOV 28	Ok(batch_id) => batch_id,	×
29	Err(e) => {	×
30	// Set the error in the error output.
31	let msg = e.to_string();	×
32	unsafe { error_out.write(cpp::duckdb_vx_error_create(msg.as_ptr().cast(), msg.len())) };	×
33	0	×
34	}
35	}
UNCOV 36	}	×

vortex-data / vortex / 16728097825

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous