From 91ebe8e6d3282d1ca042936f337674f5acca1b91 Mon Sep 17 00:00:00 2001
From: mbkkt <mbkkt@users.noreply.github.com>
Date: Thu, 2 Oct 2025 12:29:27 +0200
Subject: [PATCH] fix: Remove unused code that can confuse reader

---
 axiom/optimizer/Optimization.cpp | 2 --
 axiom/optimizer/Schema.cpp       | 1 -
 axiom/optimizer/Schema.h         | 7 +++----
 3 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/axiom/optimizer/Optimization.cpp b/axiom/optimizer/Optimization.cpp
index 561cc1e3..ea7c6c7e 100644
--- a/axiom/optimizer/Optimization.cpp
+++ b/axiom/optimizer/Optimization.cpp
@@ -1485,8 +1485,6 @@ Distribution somePartition(const RelationOpPtrVector& inputs) {
   }
 
   DistributionType distributionType;
-  distributionType.numPartitions =
-      queryCtx()->optimization()->runnerOptions().numWorkers;
   distributionType.locus = firstInput->distribution().distributionType.locus;
 
   return {distributionType, std::move(columns)};
diff --git a/axiom/optimizer/Schema.cpp b/axiom/optimizer/Schema.cpp
index ed95dd57..18a024c4 100644
--- a/axiom/optimizer/Schema.cpp
+++ b/axiom/optimizer/Schema.cpp
@@ -432,7 +432,6 @@ std::string Distribution::toString() const {
   if (!partition.empty()) {
     out << "P ";
     exprsToString(partition, out);
-    out << " " << distributionType.numPartitions << " ways";
   }
   if (!orderKeys.empty()) {
     out << " O ";
diff --git a/axiom/optimizer/Schema.h b/axiom/optimizer/Schema.h
index b05021fa..b4024eb5 100644
--- a/axiom/optimizer/Schema.h
+++ b/axiom/optimizer/Schema.h
@@ -124,14 +124,13 @@ enum class ShuffleMode : uint8_t {
   kHive,
 };
 
-/// Distribution of data. 'numPartitions' is 1 if the data is not partitioned.
+/// Distribution of data.
 /// There is copartitioning if the DistributionType is the same on both sides
 /// and both sides have an equal number of 1:1 type matched partitioning keys.
 struct DistributionType {
   bool operator==(const DistributionType& other) const = default;
 
   LocusCP locus{nullptr};
-  int32_t numPartitions{1};
   bool isGather{false};
   ShuffleMode mode{ShuffleMode::kNone};
 
@@ -199,7 +198,7 @@ struct Distribution {
   DistributionType distributionType;
 
   // Partitioning columns. The values of these columns determine which of
-  // 'numPartitions' contains any given row. This does not specify the
+  // paritions contains any given row. This does not specify the
   // partition function (e.g. Hive bucket or range partition).
   ExprVector partition;
 
@@ -225,7 +224,7 @@ struct Distribution {
   // because lineitem has an average of 4 repeats of orderkey.
   float spacing{-1};
 
-  // True if the data is replicated to 'numPartitions'.
+  // True if the data is replicated to all partitions.
   bool isBroadcast{false};
 };