Skip to content

Commit 03f940e

Browse files
committed
Make arrow::array_reader private (#1032)
1 parent d852229 commit 03f940e

File tree

3 files changed

+20
-2
lines changed

3 files changed

+20
-2
lines changed

parquet/Cargo.toml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,8 @@ arrow = { path = "../arrow", version = "7.0.0-SNAPSHOT", default-features = fals
6060
default = ["arrow", "snap", "brotli", "flate2", "lz4", "zstd", "base64"]
6161
cli = ["serde_json", "base64", "clap"]
6262
test_common = []
63+
# Experimental, unstable functionality primarily used for testing
64+
experimental = []
6365

6466
[[ bin ]]
6567
name = "parquet-read"
@@ -79,5 +81,5 @@ harness = false
7981

8082
[[bench]]
8183
name = "arrow_array_reader"
82-
required-features = ["test_common"]
84+
required-features = ["test_common", "experimental"]
8385
harness = false

parquet/src/arrow/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@
118118
//!}
119119
//! ```
120120
121-
pub mod array_reader;
121+
experimental_mod!(array_reader);
122122
pub mod arrow_array_reader;
123123
pub mod arrow_reader;
124124
pub mod arrow_writer;

parquet/src/lib.rs

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,22 @@
3535
clippy::vec_init_then_push
3636
)]
3737

38+
/// Defines a module with an experimental public API
39+
///
40+
/// The module will not be documented, and will only be public if the
41+
/// experimental feature flag is enabled
42+
///
43+
/// Experimental modules have no stability guarantees
44+
macro_rules! experimental_mod {
45+
($module:ident) => {
46+
#[cfg(feature = "experimental")]
47+
#[doc(hidden)]
48+
pub mod $module;
49+
#[cfg(not(feature = "experimental"))]
50+
mod $module;
51+
};
52+
}
53+
3854
#[macro_use]
3955
pub mod errors;
4056
pub mod basic;

0 commit comments

Comments
 (0)