Add TODOs for catalog

Author: poojanilangekar

src/storage/data_table.cpp

@@ -934,6 +934,9 @@ void DataTable::AddTileGroupWithOidForRecovery(const oid_t &tile_group_id) {
   schemas.push_back(*schema);
   std::shared_ptr<const Layout> layout = nullptr;
 
+  // The TileGroup for recovery is always added in ROW layout,
+  // This was a part of the previous design. If you are planning
+  // to change this, make sure the layout is added to the catalog
   if (default_layout_->IsRowStore()) {
     layout = default_layout_;
   } else {
@@ -1366,7 +1369,9 @@ void DataTable::ClearIndexSamples() {
 }
 
 void DataTable::SetDefaultLayout(const column_map_type &column_map) {
-  default_layout_ = std::shared_ptr<const Layout>(new const Layout(column_map));
+  // TODO Pooja: Generate layout_oid_ and add it to the catalog
+  default_layout_ = std::shared_ptr<const Layout>(
+          new const Layout(column_map));
 }
 
 const Layout& DataTable::GetDefaultLayout() const {

src/storage/temp_table.cpp

@@ -113,6 +113,7 @@ oid_t TempTable::AddDefaultTileGroup() {
       TEMPTABLE_TILEGROUP_ID + static_cast<int>(tile_groups_.size());
 
   // Create a tile group with that partitioning
+  // Its a TempTable, so we don't need to make the Layout persistent
   std::shared_ptr<storage::TileGroup> tile_group(
       AbstractTable::GetTileGroupWithLayout(
           INVALID_OID, tile_group_id, default_layout_, TEMPTABLE_DEFAULT_SIZE));