add more tests

Author: xudong963

src/query/service/src/pipelines/processors/transforms/hash_join/join_hash_table.rs

@@ -708,7 +708,7 @@ impl HashJoinState for JoinHashTable {
                         .contains(&row_ptr)
                     {
                         let mut row_state = self.hash_join_desc.right_join_desc.row_state.write();
-                        row_state.entry(row_ptr.clone()).or_insert(0_usize);
+                        row_state.entry(row_ptr).or_insert(0_usize);
                         unmatched_build_indexes.push(row_ptr);
                     }
                 }

tests/logictest/suites/base/15_query/join.test

@@ -399,6 +399,16 @@ select * from (SELECT number AS a FROM numbers(10)) x right join (SELECT number
 3
 4
 
+statement query II
+select * from (SELECT number AS a FROM numbers(1000)) x right join (SELECT number AS a FROM numbers(5))  y on x.a = y.a order by x.a;
+
+----
+0 0
+1 1
+2 2
+3 3
+4 4
+
 statement query II
 select * from numbers(10) x join (select 1::UInt64 number) y on x.number = y.number;
 

tests/logictest/suites/crdb/join

@@ -670,6 +670,28 @@ SELECT * FROM str1 LEFT OUTER JOIN str2 ON str1.s = str2.s order by str1.a;
 3 c NULL NULL
 4 D NULL NULL
 
+statement ok
+INSERT INTO str1 VALUES (1, 'a' ), (2, 'A'), (3, 'c'), (4, 'D');
+
+statement query ITIT
+select * from str1 right join str2 on str1.s = str2.s order by str2.a;
+
+----
+2 A 1 A
+2 A 1 A
+NULL NULL 2 B
+NULL NULL 3 C
+NULL NULL 4 E
+
+statement query ITIT
+select * from str1 right join str2 on false order by str2.a;
+
+----
+NULL NULL 1 A
+NULL NULL 2 B
+NULL NULL 3 C
+NULL NULL 4 E
+
 -- statement query
 -- SELECT s, str1.s, str2.s FROM str1 FULL OUTER JOIN str2 USING(s);