fix(ingress): clarify column_bool_opt None behavior, drop redundant decimal test

- column_bool_opt: spell out that None skips the column entirely rather than
  writing false — readers wanting an explicit false should call column_bool
  directly.
- Remove test_buffer_opt_decimal_some_matches_standard; the main V3 test
  already round-trips column_dec_opt("c_dec", Some(...)) against column_dec.

Author: aaaronme

questdb-rs/src/ingress/buffer.rs

@@ -871,7 +871,9 @@ impl Buffer {
     }
 
     /// Record a boolean value for the given column if the value is `Some`.
-    /// If the value is `None`, this is a no-op and the column is skipped.
+    /// If the value is `None`, this is a no-op and the column is skipped
+    /// entirely — it is **not** written as `false`. If you need an explicit
+    /// `false`, call [`Self::column_bool`] directly.
     ///
     /// This is a convenience wrapper around [`Self::column_bool`].
     ///

questdb-rs/src/tests/buffer_opt.rs

@@ -140,22 +140,3 @@ fn test_buffer_opt_array_some_binary_match() -> TestResult {
     Ok(())
 }
 
-#[test]
-fn test_buffer_opt_decimal_some_matches_standard() -> TestResult {
-    // Decimals require V3.
-    let version = ProtocolVersion::V3;
-    let mut opt_buf = Buffer::new(version);
-    opt_buf
-        .table("test")?
-        .column_dec_opt("dec", Some("123.45"))?;
-
-    let mut std_buf = Buffer::new(version);
-    std_buf.table("test")?.column_dec("dec", "123.45")?;
-
-    assert_eq!(
-        std::str::from_utf8(opt_buf.as_bytes())?,
-        std::str::from_utf8(std_buf.as_bytes())?
-    );
-
-    Ok(())
-}