Apply suggestions from code review

crates/connectors/ndc-citus/tests/query_tests.rs

@@ -254,6 +254,8 @@ mod sorting {
         insta::assert_json_snapshot!(result);
     }
 
+    // Citus sorts strings differently than PostgreSQL,
+    // so the 5th row might be different.
     #[tokio::test]
     async fn sorting_by_nested_relationship_column_with_predicate_exists() {
         let result = run_query(

crates/connectors/ndc-cockroach/tests/query_tests.rs

@@ -245,6 +245,12 @@ mod sorting {
         insta::assert_json_snapshot!(result);
     }
 
+    // CockroachDB has the opposite default for nulls ordering than PostgreSQL,
+    // making the test I've written return random results
+    // (since most tracks are not by Accept, nulls are returned
+    // first, and we limit the number of results,
+    // we are not going to get the same result here),
+    // so we skip the two other sorting with predicate tests.
     #[tokio::test]
     async fn sorting_by_relationship_count_with_predicate() {
         let result = run_query(

crates/tests/other-db-tests/src/aurora/query_tests.rs

@@ -255,6 +255,8 @@ mod sorting {
         insta::assert_json_snapshot!(result);
     }
 
+    // Aurora sorts strings differently than PostgreSQL,
+    // so the 5th row might be different.
     #[tokio::test]
     async fn sorting_by_nested_relationship_column_with_predicate_exists() {
         let result = run_query(