fix: "Memory" is misspelt (#12222)

Summary: Pull Request resolved: #12222 Reviewed By: DanielHunte Differential Revision: D69060390 Pulled By: bikramSingh91 fbshipit-source-id: 48fcc90e3a988c9f602221d13d81961195c1f98e
facebookincubator · Feb 3, 2025 · e484679 · e484679
1 parent f1dec31
commit e484679
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/velox/common/memory/SharedArbitrator.cpp b/velox/common/memory/SharedArbitrator.cpp
@@ -260,7 +260,7 @@ SharedArbitrator::SharedArbitrator(const Config& config)
       std::thread::hardware_concurrency() * memoryReclaimThreadsHwMultiplier_);
   memoryReclaimExecutor_ = std::make_unique<folly::CPUThreadPoolExecutor>(
       numReclaimThreads,
-      std::make_shared<folly::NamedThreadFactory>("MeomryReclaim"));
+      std::make_shared<folly::NamedThreadFactory>("MemoryReclaim"));
   VELOX_MEM_LOG(INFO) << "Start memory reclaim executor with "
                       << numReclaimThreads << " threads";
 

diff --git a/velox/common/memory/tests/MemoryArbitratorTest.cpp b/velox/common/memory/tests/MemoryArbitratorTest.cpp
@@ -806,7 +806,7 @@ TEST_F(MemoryReclaimerTest, orderedReclaim) {
   uint64_t reclaimableBytes{0};
   // 'expectedReclaimableUnits' is the expected allocation unit per each child
   // pool after each round of memory reclaim. And we expect the memory reclaimer
-  // always reclaim from the child with most meomry usage.
+  // always reclaim from the child with most memory usage.
   auto verify = [&](const std::vector<int>& expectedReclaimableUnits) {
     root->reclaimer()->reclaimableBytes(*root, reclaimableBytes);
     ASSERT_EQ(reclaimableBytes, totalAllocUnits * allocUnitBytes) << "total";
@@ -944,7 +944,7 @@ TEST_F(MemoryReclaimerTest, skipNonReclaimableChild) {
   uint64_t reclaimableBytes{0};
   // 'expectedReclaimableUnits' is the expected allocation unit per each child
   // pool after each round of memory reclaim. And we expect the memory reclaimer
-  // always reclaim from the child with most meomry usage.
+  // always reclaim from the child with most memory usage.
   auto verify = [&](const std::vector<int>& expectedReclaimableUnits) {
     uint64_t expectedTotalReclaimableBytes{0};
     for (int i = 0; i < numChildren; ++i) {

diff --git a/velox/exec/tests/ScaledScanControllerTest.cpp b/velox/exec/tests/ScaledScanControllerTest.cpp
@@ -121,7 +121,7 @@ TEST_F(ScaledScanControllerTest, basic) {
        3},
 
       // Test case that we can only scale up to six drivers as hit the query
-      // meomry usage ratio limit.
+      // memory usage ratio limit.
       {256 << 20,
        0.5,
        64 << 20,