Skip to content

Commit a82fd13

Browse files
committed
chore: cargo fmt
1 parent 086c41f commit a82fd13

File tree

8 files changed

+112
-72
lines changed

8 files changed

+112
-72
lines changed

src/bridge.rs

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,9 @@ pub(crate) mod ffi {
2424
pub(crate) type OpaqueCpcSketch;
2525

2626
pub(crate) fn new_opaque_cpc_sketch() -> UniquePtr<OpaqueCpcSketch>;
27-
pub(crate) fn deserialize_opaque_cpc_sketch(buf: &[u8]) -> Result<UniquePtr<OpaqueCpcSketch>>;
27+
pub(crate) fn deserialize_opaque_cpc_sketch(
28+
buf: &[u8],
29+
) -> Result<UniquePtr<OpaqueCpcSketch>>;
2830
pub(crate) fn estimate(self: &OpaqueCpcSketch) -> f64;
2931
pub(crate) fn update(self: Pin<&mut OpaqueCpcSketch>, buf: &[u8]);
3032
pub(crate) fn update_u64(self: Pin<&mut OpaqueCpcSketch>, value: u64);
@@ -81,16 +83,17 @@ pub(crate) mod ffi {
8183

8284
pub(crate) type OpaqueHhSketch;
8385

84-
pub(crate) fn new_opaque_hh_sketch(lg2_k: u8, hashset_addr: usize) -> UniquePtr<OpaqueHhSketch>;
86+
pub(crate) fn new_opaque_hh_sketch(
87+
lg2_k: u8,
88+
hashset_addr: usize,
89+
) -> UniquePtr<OpaqueHhSketch>;
8590
pub(crate) fn estimate_no_fp(
8691
self: &OpaqueHhSketch,
8792
) -> UniquePtr<CxxVector<ThinHeavyHitterRow>>;
8893
pub(crate) fn estimate_no_fn(
8994
self: &OpaqueHhSketch,
9095
) -> UniquePtr<CxxVector<ThinHeavyHitterRow>>;
91-
pub(crate) fn state(
92-
self: &OpaqueHhSketch,
93-
) -> UniquePtr<CxxVector<ThinHeavyHitterRow>>;
96+
pub(crate) fn state(self: &OpaqueHhSketch) -> UniquePtr<CxxVector<ThinHeavyHitterRow>>;
9497
pub(crate) fn update(self: Pin<&mut OpaqueHhSketch>, value: usize, weight: u64);
9598
pub(crate) fn set_weights(self: Pin<&mut OpaqueHhSketch>, total_weight: u64, weight: u64);
9699
pub(crate) fn get_total_weight(self: &OpaqueHhSketch) -> u64;

src/counters.rs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -150,29 +150,30 @@ impl KeyedMerger {
150150

151151
pub struct HeavyHitter {
152152
sketch: HhSketch,
153-
k: u64
153+
k: u64,
154154
}
155155

156156
// https://users.rust-lang.org/t/logarithm-of-integers/8506/5
157157

158158
fn log2_floor(x: u64) -> usize {
159-
const fn num_bits<T>() -> usize { std::mem::size_of::<T>() * 8 }
159+
const fn num_bits<T>() -> usize {
160+
std::mem::size_of::<T>() * 8
161+
}
160162
assert!(x > 0);
161163
num_bits::<u64>() - x.leading_zeros() as usize - 1
162164
}
163165

164166
impl HeavyHitter {
165-
166167
/// Creates a new heavy hitter sketch targeting elements in the top-k
167168
/// by reserving O(k) space.
168-
pub fn new( k: u64) -> Self {
169+
pub fn new(k: u64) -> Self {
169170
let lg2_k_with_room = log2_floor(k as u64).max(1) + 2;
170171
Self {
171172
sketch: HhSketch::new(lg2_k_with_room.try_into().unwrap()),
172-
k
173+
k,
173174
}
174175
}
175-
176+
176177
/// Serializes to base64 string with no newlines or `=` padding.
177178
pub fn serialize(&self) -> String {
178179
unimplemented!()
@@ -187,8 +188,7 @@ impl HeavyHitter {
187188
pub fn estimate(&self) -> impl Iterator<Item = (&[u8], u64)> {
188189
let mut v = self.sketch.estimate_no_fn();
189190
v.sort_by_key(|row| row.ub);
190-
v
191-
.into_iter()
191+
v.into_iter()
192192
.rev()
193193
.take(self.k as usize)
194194
.map(|row| (row.key, row.ub))

src/error.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ impl Display for DataSketchesError {
1010
fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
1111
match self {
1212
DataSketchesError::CXXError(err) => f.write_fmt(format_args!("Error: {}", err)),
13-
DataSketchesError::DecodeError(err) => f.write_fmt(format_args!("DecodeError: {}", err))
13+
DataSketchesError::DecodeError(err) => {
14+
f.write_fmt(format_args!("DecodeError: {}", err))
15+
}
1416
}
1517
}
1618
}

src/lib.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
//! `dsrs` contains bindings for a subset of [Apache DataSketches](https://github.com/apache/datasketches-cpp).
22
3-
mod error;
43
mod bridge;
54
pub mod counters;
5+
mod error;
66
pub mod stream_reducer;
77
mod wrapper;
88

9+
pub use error::DataSketchesError;
910
pub use wrapper::CpcSketch;
1011
pub use wrapper::CpcUnion;
1112
pub use wrapper::HhSketch;
1213
pub use wrapper::StaticThetaSketch;
1314
pub use wrapper::ThetaIntersection;
1415
pub use wrapper::ThetaSketch;
1516
pub use wrapper::ThetaUnion;
16-
pub use error::DataSketchesError;

src/main.rs

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -141,14 +141,13 @@ fn main() {
141141
assert!(!opt.raw, "--raw and --hh cannot be set simultaneously");
142142
assert!(!opt.merge, "--merge and --hh cannot be set simultaneously");
143143
if k == 0 {
144-
return
144+
return;
145145
}
146-
let reduced =
147-
reduce_stream(io::stdin().lock(), HeavyHitter::new(k)).expect("no io error");
146+
let reduced = reduce_stream(io::stdin().lock(), HeavyHitter::new(k)).expect("no io error");
148147
for (line, count) in reduced.estimate() {
149148
println!("{} {}", count, str::from_utf8(line).expect("valid UTF-8"));
150149
}
151-
return
150+
return;
152151
}
153152

154153
match (opt.key, opt.merge) {
@@ -221,10 +220,12 @@ mod tests {
221220
.success()
222221
.get_output()
223222
.clone();
224-
assert!(out.stderr.is_empty(), "stderr {}",
225-
str::from_utf8(&out.stderr).expect("valid UTF-8"));
226-
out
227-
.stdout
223+
assert!(
224+
out.stderr.is_empty(),
225+
"stderr {}",
226+
str::from_utf8(&out.stderr).expect("valid UTF-8")
227+
);
228+
out.stdout
228229
}
229230

230231
fn eval_bash(cmd: &str) -> Vec<u8> {
@@ -233,8 +234,11 @@ mod tests {
233234
.arg(cmd)
234235
.output()
235236
.expect("datagen process successful");
236-
assert!(out.stderr.is_empty(), "{}",
237-
str::from_utf8(&out.stderr).unwrap());
237+
assert!(
238+
out.stderr.is_empty(),
239+
"{}",
240+
str::from_utf8(&out.stderr).unwrap()
241+
);
238242
out.stdout
239243
}
240244

@@ -392,14 +396,17 @@ mod tests {
392396
}
393397

394398
fn unix_hh(k: usize) -> String {
395-
format!("sort | uniq -c | sort -rn | head -{} | sed 's/^ *//' | sort", k)
399+
format!(
400+
"sort | uniq -c | sort -rn | head -{} | sed 's/^ *//' | sort",
401+
k
402+
)
396403
}
397404

398405
fn validate_unix_hh(datagen: &str, k: usize) {
399406
let unix = unix_hh(k);
400407
let kstr = format!("{}", k);
401408
let dsrs = &["--hh", &kstr];
402-
validate_equal_cmd(datagen, dsrs, &unix);
409+
validate_equal_cmd(datagen, dsrs, &unix);
403410
}
404411

405412
#[test]

src/wrapper/cpc.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,9 @@ mod tests {
198198

199199
#[test]
200200
fn cpc_deserialization_error() {
201-
assert!(matches!(CpcSketch::deserialize(&[9, 9, 9, 9]), Err(DataSketchesError::CXXError(_))));
201+
assert!(matches!(
202+
CpcSketch::deserialize(&[9, 9, 9, 9]),
203+
Err(DataSketchesError::CXXError(_))
204+
));
202205
}
203206
}

0 commit comments

Comments
 (0)