column names are now getting full display name optionality

common/src/proto/komp_ac.table_definition.rs

@@ -251,13 +251,12 @@ pub struct PutTableDefinitionResponse {
 #[derive(serde::Serialize, serde::Deserialize)]
 #[derive(Clone, PartialEq, Eq, Hash, ::prost::Message)]
 pub struct ColumnDefinition {
-    /// Must be lowercase, start with a letter, and use only lowercase letters,
-    /// digits and underscores.
-    ///
-    /// Cannot be "id", "deleted", "created_at" or "row_revision": those are
-    /// system columns, and a data request names system and user columns in one
-    /// namespace. Any other name is free, including one ending in "\_id" -- no
-    /// column name is derived from a table name, so nothing collides.
+    /// Public display name for this column; the physical column uses a stable
+    /// server-assigned ordinal name. Preserved exactly for display. It may contain
+    /// Unicode letters, numbers, spaces, punctuation and symbols, but not
+    /// leading/trailing whitespace or control characters, and is limited to 63
+    /// UTF-8 bytes. Its canonical form must be unique within the table and cannot
+    /// claim a reserved system-column name.
     #[prost(string, tag = "1")]
     pub name: ::prost::alloc::string::String,
     /// Logical column type. Supported values (case-insensitive):