Merge branch 'main' into refactor/arrow

crates/proof-of-sql/src/base/database/mod.rs

@@ -78,4 +78,4 @@ mod filter_util_test;
 
 pub(crate) mod group_by_util;
 #[cfg(test)]
-mod group_by_util_test;
+mod group_by_util_test;

crates/proof-of-sql/src/base/database/owned_table_test_accessor.rs

@@ -6,11 +6,12 @@ use crate::base::{
     commitment::{CommitmentEvaluationProof, VecCommitmentExt},
     map::IndexMap,
 };
+use alloc::{string::String, vec::Vec};
 use bumpalo::Bump;
 use proof_of_sql_parser::Identifier;
 
 /// A test accessor that uses [`OwnedTable`] as the underlying table type.
-/// Note: this is not optimized for performance, so should not be used for benchmarks.
+/// Note: this is intended for testing and examples. It is not optimized for performance, so should not be used for benchmarks or production use-cases.
 pub struct OwnedTableTestAccessor<'a, CP: CommitmentEvaluationProof> {
     tables: IndexMap<TableRef, (OwnedTable<CP::Scalar>, usize)>,
     alloc: Bump,

crates/proof-of-sql/src/base/database/test_accessor.rs

@@ -3,10 +3,11 @@ use super::{
     SchemaAccessor, TableRef,
 };
 use crate::base::{commitment::Commitment, scalar::Curve25519Scalar};
+use alloc::vec::Vec;
 use curve25519_dalek::ristretto::RistrettoPoint;
 use proof_of_sql_parser::Identifier;
 
-/// A trait that defines the interface for a combined metadata, schema, commitment, and data accessor for unit testing purposes.
+/// A trait that defines the interface for a combined metadata, schema, commitment, and data accessor for unit testing or example purposes.
 pub trait TestAccessor<C: Commitment>:
     Clone
     + Default