Fix validation logic for behavior_version

Author: badmonster0

src/builder/analyzer.rs

@@ -716,7 +716,7 @@ impl<'a> AnalyzerContext<'a> {
                                 output_type: output_type.typ.clone(),
                             };
                             if function_exec_info.enable_cache
-                                && function_exec_info.behavior_version.is_some()
+                                && function_exec_info.behavior_version.is_none()
                             {
                                 api_bail!(
                                     "When caching is enabled, behavior version must be specified for transform op: {}",

src/ops/py_factory.rs

@@ -266,13 +266,13 @@ impl SimpleFunctionFactory for PyFunctionFactory {
                 let (enable_cache, behavior_version) =
                     Python::with_gil(|py| -> anyhow::Result<_> {
                         executor.call_method(py, "prepare", (), None)?;
-                        let enable_cache = executor.call_method(py, "enable_cache", (), None)?;
-                        let behavior_version =
-                            executor.call_method(py, "behavior_version", (), None)?;
-                        Ok((
-                            enable_cache.extract::<bool>(py)?,
-                            behavior_version.extract::<Option<u32>>(py)?,
-                        ))
+                        let enable_cache = executor
+                            .call_method(py, "enable_cache", (), None)?
+                            .extract::<bool>(py)?;
+                        let behavior_version = executor
+                            .call_method(py, "behavior_version", (), None)?
+                            .extract::<Option<u32>>(py)?;
+                        Ok((enable_cache, behavior_version))
                     })?;
                 Ok(Box::new(Arc::new(PyFunctionExecutor {
                     py_function_executor: executor,