Add comment on table creation (#2052)

### What problem does this PR solve?

_Briefly describe what this PR aims to solve. Include background context
that will help reviewers understand the purpose of the PR._

### Type of change

- [x] New Feature (non-breaking change which adds functionality)

---------

Signed-off-by: Jin Hai <haijin.chn@gmail.com>

python/infinity_http.py

@@ -294,7 +294,7 @@ def show_columns_type(self):
     def show_tables(self):
         self.get_all_tables()
         return database_result(columns=["database", "table", "type", "column_count", "block_count", "block_capacity",
-                                        "segment_count", "segment_capacity"])
+                                        "segment_count", "segment_capacity", "comment"])
 
     # index
     def create_index(

python/restart_test/test_memidx.py

@@ -207,3 +207,5 @@ def part2(infinity_obj):
             assert len(idx2_files_in_dir) == 1
 
         part2()
+
+        infinity_runner.clear()

python/test_pysdk/test_table.py

@@ -514,7 +514,7 @@ def _test_show_tables(self, suffix):
             print(res)
             # check the polars dataframe
             assert res.columns == ["database", "table", "type", "column_count", "block_count", "block_capacity",
-                                   "segment_count", "segment_capacity"]
+                                   "segment_count", "segment_capacity", "comment"]
 
     def _test_create_varchar_table(self, suffix):
         """

src/executor/operator/physical_drop_index.cpp

@@ -35,15 +35,15 @@ bool PhysicalDropIndex::Execute(QueryContext *query_context, OperatorState *oper
     auto txn = query_context->GetTxn();
     Status status = txn->DropIndexByName(*schema_name_, *table_name_, *index_name_, conflict_type_);
 
-    if(!status.ok()) {
+    if (!status.ok()) {
         operator_state->status_ = status;
     }
 
     // Generate the result
     Vector<SharedPtr<ColumnDef>> column_defs = {
         MakeShared<ColumnDef>(0, MakeShared<DataType>(LogicalType::kInteger), "OK", std::set<ConstraintType>())};
 
-    auto result_table_def_ptr = MakeShared<TableDef>(MakeShared<String>("default_db"), MakeShared<String>("Tables"), column_defs);
+    auto result_table_def_ptr = TableDef::Make(MakeShared<String>("default_db"), MakeShared<String>("Tables"), nullptr, column_defs);
     output_ = MakeShared<DataTable>(result_table_def_ptr, TableType::kDataTable);
     operator_state->SetComplete();
     return true;

src/executor/operator/physical_drop_schema.cpp

@@ -36,15 +36,16 @@ void PhysicalDropSchema::Init() {}
 bool PhysicalDropSchema::Execute(QueryContext *query_context, OperatorState *operator_state) {
     auto txn = query_context->GetTxn();
     Status status = txn->DropDatabase(*schema_name_, conflict_type_);
-    if(!status.ok()) {
+    if (!status.ok()) {
         operator_state->status_ = status;
     }
 
     // Generate the result
     Vector<SharedPtr<ColumnDef>> column_defs = {
         MakeShared<ColumnDef>(0, MakeShared<DataType>(LogicalType::kInteger), "OK", std::set<ConstraintType>())};
 
-    auto result_table_def_ptr = MakeShared<TableDef>(MakeShared<String>("default_db"), MakeShared<String>("Tables"), column_defs);
+    auto result_table_def_ptr =
+        TableDef::Make(MakeShared<String>("default_db"), MakeShared<String>("Tables"), nullptr, column_defs);
     output_ = MakeShared<DataTable>(result_table_def_ptr, TableType::kDataTable);
     operator_state->SetComplete();
     return true;

src/executor/operator/physical_drop_table.cpp

@@ -41,15 +41,15 @@ bool PhysicalDropTable::Execute(QueryContext *query_context, OperatorState *oper
         cache_mgr->DropTable(*schema_name_, *table_name_);
     }
 
-    if(!status.ok()) {
+    if (!status.ok()) {
         operator_state->status_ = status;
     }
 
     // Generate the result
     Vector<SharedPtr<ColumnDef>> column_defs = {
         MakeShared<ColumnDef>(0, MakeShared<DataType>(LogicalType::kInteger), "OK", std::set<ConstraintType>())};
 
-    auto result_table_def_ptr = MakeShared<TableDef>(MakeShared<String>("default_db"), MakeShared<String>("Tables"), column_defs);
+    auto result_table_def_ptr = TableDef::Make(MakeShared<String>("default_db"), MakeShared<String>("Tables"), nullptr, column_defs);
     output_ = MakeShared<DataTable>(result_table_def_ptr, TableType::kDataTable);
     operator_state->SetComplete();
     return true;

src/executor/operator/physical_insert.cpp

@@ -47,7 +47,8 @@ bool PhysicalInsert::Execute(QueryContext *query_context, OperatorState *operato
     SizeT column_count = value_list_[0].size();
     SizeT table_collection_column_count = table_entry_->ColumnCount();
     if (column_count != table_collection_column_count) {
-        String error_message = fmt::format("Insert values count{} isn't matched with table column count{}.", column_count, table_collection_column_count);
+        String error_message =
+            fmt::format("Insert values count{} isn't matched with table column count{}.", column_count, table_collection_column_count);
         UnrecoverableError(error_message);
     }
 
@@ -86,7 +87,8 @@ bool PhysicalInsert::Execute(QueryContext *query_context, OperatorState *operato
     if (operator_state == nullptr) {
         // Generate the result table
         Vector<SharedPtr<ColumnDef>> column_defs;
-        SharedPtr<TableDef> result_table_def_ptr = MakeShared<TableDef>(MakeShared<String>("default_db"), MakeShared<String>("Tables"), column_defs);
+        SharedPtr<TableDef> result_table_def_ptr =
+            TableDef::Make(MakeShared<String>("default_db"), MakeShared<String>("Tables"), nullptr, column_defs);
         output_ = MakeShared<DataTable>(result_table_def_ptr, TableType::kDataTable);
         output_->SetResultMsg(std::move(result_msg));
     } else {