Skip to content

Commit f32890f

Browse files
zhjwpkuwgtmac
andauthored
Polish comment per review.
Co-authored-by: Gang Wu <[email protected]>
1 parent 401948b commit f32890f

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/iceberg/expression/literal.h

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -130,12 +130,12 @@ class ICEBERG_EXPORT Literal : public util::Formattable {
130130

131131
bool operator==(const Literal& other) const;
132132

133-
/// \brief Compare two PrimitiveLiterals. Both literals must have the same type.
134-
/// \param other The other PrimitiveLiteral to compare with.
133+
/// \brief Compare two literals of the same primitive type.
134+
/// \param other The other literal to compare with.
135135
/// \return The comparison result as std::partial_ordering. If either side is AboveMax,
136-
/// BelowMin or null, the result is unordered.
137-
/// Note: This comparison can not be used for sorting literal vectors containing
138-
/// AboveMax, BelowMin or null values.
136+
/// BelowMin or Null, the result is unordered.
137+
/// Note: This comparison cannot be used for sorting literals if any literal is
138+
/// AboveMax, BelowMin or Null.
139139
std::partial_ordering operator<=>(const Literal& other) const;
140140

141141
/// Check if this literal represents a value above the maximum allowed value

0 commit comments

Comments
 (0)