Apply suggestions from code review

Co-authored-by: Mario Montoya <mamcx@elmalabarista.com>
Signed-off-by: joshua-spacetime <josh@clockworklabs.io>

Author: joshua-spacetime

crates/core/src/subscription/module_subscription_manager.rs

@@ -1118,11 +1118,11 @@ impl SubscriptionManager {
                 &row.elements[edge.lhs_join_col.idx()],
             )
             // This read should always succeed, and it's a bug if it doesn't.
-            .unwrap()
+            .expect("This read should always succeed, and it's a bug if it doesn't")
             .next()
             .map(|row| {
                 // Similarly this read should always succeed
-                row.read_col(edge.rhs_col).unwrap()
+                row.read_col(edge.rhs_col).expect("This read should always succeed, and it's a bug if it doesn't")
             })
         }
 

crates/subscription/src/lib.rs

@@ -300,7 +300,7 @@ impl std::fmt::Display for TableName {
 pub struct JoinEdge {
     /// The [`TableId`] for `a`
     pub lhs_table: TableId,
-    /// The [`TableId`] for `a`
+    /// The [`TableId`] for `b`
     pub rhs_table: TableId,
     /// The [`ColId`] for `a.id`
     pub lhs_join_col: ColId,