fix typo, macro, and use proper header

Author: yhmtsai

core/base/dense_cache_accessor.hpp

@@ -6,7 +6,6 @@
 #define GKO_CORE_BASE_DENSE_CACHE_ACCESSOR_HPP_
 
 
-#include <iostream>
 #include <map>
 #include <string>
 

core/distributed/vector_cache_accessor.hpp

@@ -6,15 +6,14 @@
 #define GKO_CORE_DISTRIBUTED_VECTOR_CACHE_ACCESSOR_HPP_
 
 
-#include <iostream>
-#include <map>
-#include <string>
-
+#include <ginkgo/config.hpp>
 #include <ginkgo/core/base/array.hpp>
-#include <ginkgo/core/base/executor.hpp>
 #include <ginkgo/core/distributed/vector_cache.hpp>
 
 
+#if GINKGO_BUILD_MPI
+
+
 namespace gko {
 namespace experimental {
 namespace distributed {
@@ -25,7 +24,9 @@ namespace detail {
 class GenericVectorCacheAccessor {
 public:
     // access to the workspace
-    static const array<char>& get_workspace(const GenericVectorCache& cache);
+    static const array<char>& get_workspace(
+        const gko::experimental::distributed::detail::GenericVectorCache&
+            cache);
 };
 
 
@@ -35,4 +36,5 @@ class GenericVectorCacheAccessor {
 }  // namespace gko
 
 
+#endif
 #endif  // GKO_CORE_DISTRIBUTED_VECTOR_CACHE_ACCESSOR_HPP_

core/test/base/dense_cache.cpp

@@ -303,7 +303,7 @@ TYPED_TEST(GenericDenseCache, GenericCanInitWithSizeAndType)
     GKO_ASSERT_EQUAL_DIMENSIONS(second_buffer->get_size(), this->size);
     ASSERT_EQ(second_buffer->get_executor(), this->ref);
     if (sizeof(another_type) > sizeof(value_type)) {
-        // the requring workspace will be bigger if the type is larger.
+        // the requiring workspace will be bigger if the type is larger.
         ASSERT_NE(
             array_ptr,
             generic_accessor::get_workspace(this->cache).get_const_data());

core/test/mpi/distributed/vector_cache.cpp

@@ -480,7 +480,7 @@ TYPED_TEST(GenericVectorCache, GenericCanInitWithSizeAndType)
     ASSERT_EQ(second_buffer->get_communicator(), this->comm);
     ASSERT_EQ(second_buffer->get_executor(), this->ref);
     if (sizeof(another_type) > sizeof(value_type)) {
-        // the requring workspace will be bigger if the type is larger.
+        // the requiring workspace will be bigger if the type is larger.
         ASSERT_NE(
             array_ptr,
             generic_accessor::get_workspace(this->cache).get_const_data());