chore(cubesql): Add comment about async generate_sql_for_expr

mcheshkov · mcheshkov · commit 3f3d9a4b9928 · 2025-01-13T15:27:38.000+02:00
diff --git a/rust/cubesql/cubesql/src/compile/engine/df/wrapper.rs b/rust/cubesql/cubesql/src/compile/engine/df/wrapper.rs
@@ -1406,6 +1406,8 @@ impl CubeScanWrapperNode {
             .map_err(|e| DataFusionError::Internal(format!("Can't generate SQL for type: {}", e)))
     }
 
+    /// This function is async to be able to call to JS land,
+    /// in case some SQL generation could not be done through Jinja
     pub fn generate_sql_for_expr<'ctx>(
         plan: Arc<Self>,
         mut sql_query: SqlQuery,

Original file line number	Diff line number	Diff line change
`@@ -1406,6 +1406,8 @@ impl CubeScanWrapperNode {`
`1406`	`1406`	`.map_err(\|e\| DataFusionError::Internal(format!("Can't generate SQL for type: {}", e)))`
`1407`	`1407`	`}`
`1408`	`1408`
	`1409`	`+ /// This function is async to be able to call to JS land,`
	`1410`	`+ /// in case some SQL generation could not be done through Jinja`
`1409`	`1411`	`pub fn generate_sql_for_expr<'ctx>(`
`1410`	`1412`	`plan: Arc<Self>,`
`1411`	`1413`	`mut sql_query: SqlQuery,`