Update TODO doc strings with current status

Author: ntjohnson1

src/common/data_type.rs

@@ -56,7 +56,7 @@ pub enum RexType {
 /// and manageable location. Therefore this structure exists
 /// to map those types and provide a simple place for developers
 /// to map types from one system to another.
-// TODO: This looks like this needs pyo3 tracking so leaving unfrozen for now
+// TODO: pyclass frozen
 #[derive(Debug, Clone)]
 #[pyclass(name = "DataTypeMap", module = "datafusion.common", subclass)]
 pub struct DataTypeMap {

src/common/schema.rs

@@ -33,7 +33,7 @@ use crate::sql::logical::PyLogicalPlan;
 
 use super::{data_type::DataTypeMap, function::SqlFunction};
 
-// TODO I think the get/set prohibits frozen
+// TODO: pyclass frozen
 #[pyclass(name = "SqlSchema", module = "datafusion.common", subclass)]
 #[derive(Debug, Clone)]
 pub struct SqlSchema {
@@ -47,6 +47,7 @@ pub struct SqlSchema {
     pub functions: Vec<SqlFunction>,
 }
 
+// TODO: pyclass frozen
 #[pyclass(name = "SqlTable", module = "datafusion.common", subclass)]
 #[derive(Debug, Clone)]
 pub struct SqlTable {
@@ -91,6 +92,7 @@ impl SqlTable {
     }
 }
 
+// TODO: pyclass frozen
 #[pyclass(name = "SqlView", module = "datafusion.common", subclass)]
 #[derive(Debug, Clone)]
 pub struct SqlView {

src/expr/conditional_expr.rs

@@ -19,7 +19,9 @@ use crate::{errors::PyDataFusionResult, expr::PyExpr};
 use datafusion::logical_expr::conditional_expressions::CaseBuilder;
 use pyo3::prelude::*;
 
-// TODO: Can't make frozen because needs to mutate case_builder
+// TODO: pyclass frozen
+// Mutates CaseBuilder might need CaseBuilder derive clone upstream
+// since when basically clones
 #[pyclass(name = "CaseBuilder", module = "datafusion.expr", subclass)]
 pub struct PyCaseBuilder {
     pub case_builder: CaseBuilder,