search

cmu-db / bustub

The BusTub Relational Database Management System (Educational)
https://15445.courses.cs.cmu.edu
MIT License
4.09k stars 1.8k forks source link

gradescope reporting format issue on gtest files in Project 3 #238

Closed Anarion-zuo closed 2 years ago

Anarion-zuo commented 2 years ago

I got this after submitting project 3 on gradescope


 Checking: /autograder/bustub/build/googletest-src/googlemock/src/gmock_main.cc
 Checking: /autograder/bustub/build/googletest-src/googlemock/src/gmock-all.cc
 Checking: /autograder/bustub/build/googletest-src/googletest/src/gtest_main.cc
 Checking: /autograder/bustub/build/googletest-src/googletest/src/gtest-all.cc
 Checking: /autograder/bustub/src/buffer/buffer_pool_manager_instance.cpp
 Checking: /autograder/bustub/src/buffer/clock_replacer.cpp
 Checking: /autograder/bustub/src/buffer/lru_replacer.cpp
 Checking: /autograder/bustub/src/buffer/parallel_buffer_pool_manager.cpp
 Checking: /autograder/bustub/src/catalog/column.cpp
 Checking: /autograder/bustub/src/catalog/schema.cpp
 Checking: /autograder/bustub/src/catalog/table_generator.cpp
 Checking: /autograder/bustub/src/common/config.cpp
 Checking: /autograder/bustub/src/common/util/string_util.cpp
 Checking: /autograder/bustub/src/concurrency/lock_manager.cpp
 Checking: /autograder/bustub/src/concurrency/transaction_manager.cpp
 Checking: /autograder/bustub/src/container/hash/extendible_hash_table.cpp
 Checking: /autograder/bustub/src/container/hash/linear_probe_hash_table.cpp
 Checking: /autograder/bustub/src/execution/aggregation_executor.cpp
 Checking: /autograder/bustub/src/execution/delete_executor.cpp
 Checking: /autograder/bustub/src/execution/distinct_executor.cpp
 Checking: /autograder/bustub/src/execution/executor_factory.cpp
 Checking: /autograder/bustub/src/execution/hash_join_executor.cpp
 Checking: /autograder/bustub/src/execution/index_scan_executor.cpp
 Checking: /autograder/bustub/src/execution/insert_executor.cpp
 Checking: /autograder/bustub/src/execution/limit_executor.cpp
 Checking: /autograder/bustub/src/execution/mock_scan_executor.cpp
 Checking: /autograder/bustub/src/execution/nested_index_join_executor.cpp
 Checking: /autograder/bustub/src/execution/nested_loop_join_executor.cpp
 Checking: /autograder/bustub/src/execution/seq_scan_executor.cpp
 Checking: /autograder/bustub/src/execution/update_executor.cpp
 Checking: /autograder/bustub/src/recovery/checkpoint_manager.cpp
 Checking: /autograder/bustub/src/recovery/log_manager.cpp
 Checking: /autograder/bustub/src/recovery/log_recovery.cpp
 Checking: /autograder/bustub/src/storage/disk/disk_manager.cpp
 Checking: /autograder/bustub/src/storage/disk/disk_manager_memory.cpp
 Checking: /autograder/bustub/src/storage/index/b_plus_tree.cpp
 Checking: /autograder/bustub/src/storage/index/b_plus_tree_index.cpp
 Checking: /autograder/bustub/src/storage/index/extendible_hash_table_index.cpp
 Checking: /autograder/bustub/src/storage/index/index_iterator.cpp
 Checking: /autograder/bustub/src/storage/index/linear_probe_hash_table_index.cpp
 Checking: /autograder/bustub/src/storage/page/b_plus_tree_internal_page.cpp
 Checking: /autograder/bustub/src/storage/page/b_plus_tree_leaf_page.cpp
 Checking: /autograder/bustub/src/storage/page/b_plus_tree_page.cpp
 Checking: /autograder/bustub/src/storage/page/hash_table_block_page.cpp
 Checking: /autograder/bustub/src/storage/page/hash_table_bucket_page.cpp
 Checking: /autograder/bustub/src/storage/page/hash_table_directory_page.cpp
 Checking: /autograder/bustub/src/storage/page/hash_table_header_page.cpp
 Checking: /autograder/bustub/src/storage/page/header_page.cpp
 Checking: /autograder/bustub/src/storage/page/table_page.cpp
 Checking: /autograder/bustub/src/storage/table/table_heap.cpp
 Checking: /autograder/bustub/src/storage/table/table_iterator.cpp
 Checking: /autograder/bustub/src/storage/table/tuple.cpp
 Checking: /autograder/bustub/src/type/bigint_type.cpp
 Checking: /autograder/bustub/src/type/boolean_type.cpp
 Checking: /autograder/bustub/src/type/decimal_type.cpp
 Checking: /autograder/bustub/src/type/integer_parent_type.cpp
 Checking: /autograder/bustub/src/type/integer_type.cpp
 Checking: /autograder/bustub/src/type/smallint_type.cpp
 Checking: /autograder/bustub/src/type/timestamp_type.cpp
 Checking: /autograder/bustub/src/type/tinyint_type.cpp
 Checking: /autograder/bustub/src/type/type.cpp
 Checking: /autograder/bustub/src/type/value.cpp
 Checking: /autograder/bustub/src/type/varlen_type.cpp
 Checking: /autograder/bustub/third_party/murmur3/MurmurHash3.cpp
 Checking: /autograder/bustub/test/type/type_test.cpp
 Checking: /autograder/bustub/test/table/tuple_test.cpp
 Checking: /autograder/bustub/test/storage/tmp_tuple_page_test.cpp
 Checking: /autograder/bustub/test/storage/grading_b_plus_tree_memory_test.cpp
 Checking: /autograder/bustub/test/storage/grading_b_plus_tree_checkpoint_2_sequential_test.cpp
 Checking: /autograder/bustub/test/storage/grading_b_plus_tree_checkpoint_1_test.cpp
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1358:11: error: The left operand of '==' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult,-warnings-as-errors]
  if (lhs == rhs) {
          ^
/autograder/bustub/test/storage/tmp_tuple_page_test.cpp:32:3: note: Control jumps to 'case 0:'  at line 32
  ASSERT_EQ(*reinterpret_cast<page_id_t *>(data), page_id);
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1877:31: note: expanded from macro 'ASSERT_EQ'
#define ASSERT_EQ(val1, val2) GTEST_ASSERT_EQ(val1, val2)
                              ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1861:3: note: expanded from macro 'GTEST_ASSERT_EQ'
  ASSERT_PRED_FORMAT2(::testing::internal::EqHelper::Compare, val1, val2)
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:148:3: note: expanded from macro 'ASSERT_PRED_FORMAT2'
  GTEST_PRED_FORMAT2_(pred_format, v1, v2, GTEST_FATAL_FAILURE_)
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:134:3: note: expanded from macro 'GTEST_PRED_FORMAT2_'
  GTEST_ASSERT_(pred_format(#v1, #v2, v1, v2), on_failure)
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:78:3: note: expanded from macro 'GTEST_ASSERT_'
  GTEST_AMBIGUOUS_ELSE_BLOCKER_                                 \
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/internal/gtest-port.h:637:3: note: expanded from macro 'GTEST_AMBIGUOUS_ELSE_BLOCKER_'
  switch (0)                          \
  ^
/autograder/bustub/test/storage/tmp_tuple_page_test.cpp:32:3: note: Calling 'EqHelper::Compare'
  ASSERT_EQ(*reinterpret_cast<page_id_t *>(data), page_id);
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1877:31: note: expanded from macro 'ASSERT_EQ'
#define ASSERT_EQ(val1, val2) GTEST_ASSERT_EQ(val1, val2)
                              ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1861:23: note: expanded from macro 'GTEST_ASSERT_EQ'
  ASSERT_PRED_FORMAT2(::testing::internal::EqHelper::Compare, val1, val2)
                      ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:148:23: note: expanded from macro 'ASSERT_PRED_FORMAT2'
  GTEST_PRED_FORMAT2_(pred_format, v1, v2, GTEST_FATAL_FAILURE_)
                      ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:134:17: note: expanded from macro 'GTEST_PRED_FORMAT2_'
  GTEST_ASSERT_(pred_format(#v1, #v2, v1, v2), on_failure)
                ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:79:52: note: expanded from macro 'GTEST_ASSERT_'
  if (const ::testing::AssertionResult gtest_ar = (expression)) \
                                                   ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1377:56: note: Passing value via 3rd parameter 'lhs'
    return CmpHelperEQ(lhs_expression, rhs_expression, lhs, rhs);
                                                       ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1377:12: note: Calling 'CmpHelperEQ<int, int>'
    return CmpHelperEQ(lhs_expression, rhs_expression, lhs, rhs);
           ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1358:11: note: The left operand of '==' is a garbage value
  if (lhs == rhs) {
          ^

 Checking: /autograder/bustub/test/storage/grading_b_plus_tree_checkpoint_2_concurrent_test.cpp
 Checking: /autograder/bustub/test/storage/grading_b_plus_tree_bench_test.cpp
 Checking: /autograder/bustub/test/storage/b_plus_tree_insert_test.cpp
 Checking: /autograder/bustub/test/storage/grading_tmp_tuple_page_test.cpp
 Checking: /autograder/bustub/test/storage/b_plus_tree_concurrent_test.cpp
 Checking: /autograder/bustub/test/recovery/recovery_test.cpp
 Checking: /autograder/bustub/test/primer/starter_test.cpp
 Checking: /autograder/bustub/test/primer/grading_starter_test.cpp
 Checking: /autograder/bustub/test/execution/grading_update_executor_test.cpp
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1358:11: error: The left operand of '==' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult,-warnings-as-errors]
  if (lhs == rhs) {
          ^
/autograder/bustub/test/storage/grading_tmp_tuple_page_test.cpp:101:3: note: Control jumps to 'case 0:'  at line 101
  ASSERT_EQ(*reinterpret_cast<page_id_t *>(data), page_id);
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1877:31: note: expanded from macro 'ASSERT_EQ'
#define ASSERT_EQ(val1, val2) GTEST_ASSERT_EQ(val1, val2)
                              ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1861:3: note: expanded from macro 'GTEST_ASSERT_EQ'
  ASSERT_PRED_FORMAT2(::testing::internal::EqHelper::Compare, val1, val2)
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:148:3: note: expanded from macro 'ASSERT_PRED_FORMAT2'
  GTEST_PRED_FORMAT2_(pred_format, v1, v2, GTEST_FATAL_FAILURE_)
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:134:3: note: expanded from macro 'GTEST_PRED_FORMAT2_'
  GTEST_ASSERT_(pred_format(#v1, #v2, v1, v2), on_failure)
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:78:3: note: expanded from macro 'GTEST_ASSERT_'
  GTEST_AMBIGUOUS_ELSE_BLOCKER_                                 \
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/internal/gtest-port.h:637:3: note: expanded from macro 'GTEST_AMBIGUOUS_ELSE_BLOCKER_'
  switch (0)                          \
  ^
/autograder/bustub/test/storage/grading_tmp_tuple_page_test.cpp:101:3: note: Calling 'EqHelper::Compare'
  ASSERT_EQ(*reinterpret_cast<page_id_t *>(data), page_id);
  ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1877:31: note: expanded from macro 'ASSERT_EQ'
#define ASSERT_EQ(val1, val2) GTEST_ASSERT_EQ(val1, val2)
                              ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1861:23: note: expanded from macro 'GTEST_ASSERT_EQ'
  ASSERT_PRED_FORMAT2(::testing::internal::EqHelper::Compare, val1, val2)
                      ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:148:23: note: expanded from macro 'ASSERT_PRED_FORMAT2'
  GTEST_PRED_FORMAT2_(pred_format, v1, v2, GTEST_FATAL_FAILURE_)
                      ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:134:17: note: expanded from macro 'GTEST_PRED_FORMAT2_'
  GTEST_ASSERT_(pred_format(#v1, #v2, v1, v2), on_failure)
                ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest_pred_impl.h:79:52: note: expanded from macro 'GTEST_ASSERT_'
  if (const ::testing::AssertionResult gtest_ar = (expression)) \
                                                   ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1377:56: note: Passing value via 3rd parameter 'lhs'
    return CmpHelperEQ(lhs_expression, rhs_expression, lhs, rhs);
                                                       ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1377:12: note: Calling 'CmpHelperEQ<int, int>'
    return CmpHelperEQ(lhs_expression, rhs_expression, lhs, rhs);
           ^
/autograder/bustub/build/googletest-src/googletest/include/gtest/gtest.h:1358:11: note: The left operand of '==' is a garbage value
  if (lhs == rhs) {
          ^

 Checking: /autograder/bustub/test/execution/grading_nested_loop_join_executor_test.cpp
 Checking: /autograder/bustub/test/execution/grading_limit_executor_test.cpp
 Checking: /autograder/bustub/test/concurrency/transaction_test.cpp
 Checking: /autograder/bustub/test/execution/grading_executor_benchmark_test.cpp
 Checking: /autograder/bustub/test/recovery/grading_recovery_test.cpp
 Checking: /autograder/bustub/test/concurrency/grading_lock_manager_3_test.cpp
 Checking: /autograder/bustub/test/buffer/lru_replacer_test.cpp
 Checking: /autograder/bustub/test/buffer/parallel_buffer_pool_manager_test.cpp
 Checking: /autograder/bustub/test/buffer/grading_parallel_buffer_pool_manager_test.cpp
 Checking: /autograder/bustub/test/buffer/grading_lru_replacer_test.cpp
 Checking: /autograder/bustub/test/execution/grading_executor_integrated_test.cpp
 Checking: /autograder/bustub/test/storage/disk_manager_test.cpp
 Checking: /autograder/bustub/test/buffer/buffer_pool_manager_instance_test.cpp
 Checking: /autograder/bustub/test/execution/grading_sequential_scan_executor_test.cpp
 Checking: /autograder/bustub/test/concurrency/grading_lock_manager_1_test.cpp
 Checking: /autograder/bustub/test/storage/b_plus_tree_print_test.cpp
 Checking: /autograder/bustub/test/buffer/clock_replacer_test.cpp
 Checking: /autograder/bustub/test/execution/grading_distinct_executor_test.cpp
 Checking: /autograder/bustub/test/buffer/grading_buffer_pool_manager_instance_test.cpp
 Checking: /autograder/bustub/test/concurrency/grading_lock_manager_2_test.cpp
 Checking: /autograder/bustub/test/concurrency/grading_transaction_test.cpp
 Checking: /autograder/bustub/test/storage/grading_b_plus_tree_test.cpp
 Checking: /autograder/bustub/test/buffer/grading_leaderboard_test.cpp
 Checking: /autograder/bustub/test/container/grading_hash_table_verification_test.cpp
 Checking: /autograder/bustub/test/concurrency/grading_rollback_test.cpp
 Checking: /autograder/bustub/test/container/hash_table_page_test.cpp
 Checking: /autograder/bustub/test/concurrency/lock_manager_test.cpp
 Checking: /autograder/bustub/test/storage/b_plus_tree_delete_test.cpp
 Checking: /autograder/bustub/test/container/grading_hash_table_concurrent_test.cpp
 Checking: /autograder/bustub/test/container/grading_hash_table_page_test.cpp
 Checking: /autograder/bustub/test/concurrency/grading_lock_manager_detection_test.cpp
 Checking: /autograder/bustub/test/container/grading_hash_table_leaderboard_test.cpp
 Checking: /autograder/bustub/test/common/rwlatch_test.cpp
 Checking: /autograder/bustub/test/container/grading_hash_table_scale_test.cpp
 Checking: /autograder/bustub/test/container/grading_hash_table_test.cpp
 Checking: /autograder/bustub/test/container/hash_table_test.cpp
 Checking: /autograder/bustub/test/execution/grading_aggregation_executor_test.cpp
 Checking: /autograder/bustub/test/catalog/catalog_test.cpp
 Checking: /autograder/bustub/test/execution/grading_insert_executor_test.cpp
 Checking: /autograder/bustub/test/execution/grading_delete_executor_test.cpp
 Checking: /autograder/bustub/test/execution/executor_test.cpp
 Checking: /autograder/bustub/test/execution/grading_hash_join_executor_test.cppThe files that failed were:
[u'/autograder/bustub/test/storage/tmp_tuple_page_test.cpp',
 u'/autograder/bustub/test/storage/grading_tmp_tuple_page_test.cpp']
Note that a failing .h file will fail all the .cpp files that include it.

CMakeFiles/check-clang-tidy.dir/build.make:57: recipe for target 'CMakeFiles/check-clang-tidy' failed
CMakeFiles/Makefile2:164: recipe for target 'CMakeFiles/check-clang-tidy.dir/all' failed
CMakeFiles/Makefile2:171: recipe for target 'CMakeFiles/check-clang-tidy.dir/rule' failed
Makefile:214: recipe for target 'check-clang-tidy' failed

make[3]: *** [CMakeFiles/check-clang-tidy] Error 1
make[2]: *** [CMakeFiles/check-clang-tidy.dir/all] Error 2
make[1]: *** [CMakeFiles/check-clang-tidy.dir/rule] Error 2
make: *** [check-clang-tidy] Error 2
fengbear commented 2 years ago

I also got this after submitting project 2 on gradescope，I don‘t know why

fengbear commented 2 years ago

I also got this after submitting project 2 on gradescope，I don‘t know why

I have solved it

Anarion-zuo commented 2 years ago

I also got this after submitting project 2 on gradescope，I don‘t know why

I have solved it

How did you do it?

wtsclwq commented 2 years ago

@fengbear How did you solve it?

Rachelint commented 2 years ago

You can see this issue.

skyzh commented 2 years ago

See https://github.com/cmu-db/bustub/issues/227 for more information.