WIP:

timescale · Sep 3, 2024 · de3075b · de3075b
1 parent cfa78bf
commit de3075b
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 8 deletions.
diff --git a/extension/src/stats_agg.rs b/extension/src/stats_agg.rs
@@ -1644,13 +1644,23 @@ mod tests {
 
             let control = state.unwrap();
             let buffer = stats1d_trans_serialize(Inner::from(control.clone()).internal().unwrap());
-            let buffer = pgrx::varlena::varlena_to_byte_slice(buffer.0.cast_mut_ptr());
+            let slice_for_test =
+                pgrx::varlena::varlena_to_byte_slice(buffer.0.cast_mut_ptr::<pg_sys::varlena>());
+            println!(
+                "debug serializer output after return:\n{:?}",
+                slice_for_test
+            );
 
-            let expected = pgrx::varlena::rust_byte_slice_to_bytea(buffer);
-            let new_state =
-                stats1d_trans_deserialize_inner(bytea(pg_sys::Datum::from(expected.as_ptr())));
+            // let expected = pgrx::varlena::rust_byte_slice_to_bytea(buffer);
+            // let new_state =
+            //     stats1d_trans_deserialize_inner(bytea(pg_sys::Datum::from(expected.as_ptr())));
+            let new_state = stats1d_trans_deserialize_inner(buffer);
 
-            assert_eq!(&*new_state, &*control);
+            assert_eq!(
+                &*new_state, &*control,
+                "unexpected difference in bytes output, got:\n{:?}\nexpected:\n{:?}",
+                &*new_state, &*control
+            );
         }
     }
 

diff --git a/extension/src/type_builder.rs b/extension/src/type_builder.rs
@@ -362,7 +362,11 @@ macro_rules! do_serialize {
                 let len = writer.position().try_into().expect("serialized size too large");
                 ::pgrx::set_varsize(writer.get_mut().as_mut_ptr() as *mut _, len);
             }
-            $crate::raw::bytea::from(pg_sys::Datum::from(writer.into_inner().as_mut_ptr()))
+
+            let raw_bytes = writer.into_inner();
+            println!("debug serializer output:\n{:?}", raw_bytes);
+            crate::raw::bytea::from(pg_sys::Datum::from(raw_bytes.as_mut_ptr()))
+            // $crate::raw::bytea::from(pg_sys::Datum::from(writer.into_inner().as_mut_ptr()))
         }
     };
 }

diff --git a/tools/build b/tools/build
@@ -104,9 +104,9 @@ while [ $# -gt 0 ]; do
             $nop cargo fetch
             export RUSTC_BOOTSTRAP=1
             export __CARGO_TEST_CHANNEL_OVERRIDE_DO_NOT_USE_THIS=nightly
-            export RUSTFLAGS='-Z sanitizer=memory'
+            # export RUSTFLAGS='-Z sanitizer=memory'
             rustup component add rust-src --toolchain 1.74.0-x86_64-unknown-linux-gnu
-            $nop cargo test -Zbuild-std --profile $profile --target --target x86_64-unknown-linux-gnu --features "$pg pg_test" --no-default-features -- stats_agg::tests
+            $nop cargo test -Zbuild-std --profile $profile --target --target x86_64-unknown-linux-gnu --features "$pg pg_test" --no-default-features -- --nocapture stats_agg::tests
             ;;
 
         install)