fix: improve binary compatibility check

Author: RaphDal

questdb-rs/src/ingress/buffer.rs

@@ -1056,7 +1056,7 @@ impl Buffer {
         self.write_column_key(name)?;
         value.serialize(
             &mut self.output,
-            self.protocol_version == ProtocolVersion::V2,
+            self.protocol_version.supports_binary_encoding(),
         )?;
         Ok(self)
     }

questdb-rs/src/ingress/mod.rs

@@ -91,6 +91,23 @@ pub enum ProtocolVersion {
     V2 = 2,
 }
 
+impl ProtocolVersion {
+    /// Returns `true` if this protocol version supports binary-encoded column values.
+    ///
+    /// # Examples
+    ///
+    /// ```
+    /// use questdb::ingress::ProtocolVersion;
+    ///
+    /// assert_eq!(ProtocolVersion::V1.supports_binary_encoding(), false);
+    /// assert_eq!(ProtocolVersion::V2.supports_binary_encoding(), true);
+    /// ```
+    #[inline]
+    pub fn supports_binary_encoding(self) -> bool {
+        self != ProtocolVersion::V1
+    }
+}
+
 impl Display for ProtocolVersion {
     fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
         match self {