lowka commented on code in PR #2671:
URL: https://github.com/apache/ignite-3/pull/2671#discussion_r1354596134
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/ExpressionFactoryImpl.java:
##########
@@ -93,9 +92,6 @@
public class ExpressionFactoryImpl<RowT> implements ExpressionFactory<RowT> {
private static final int CACHE_SIZE = 1024;
- /** Placeholder for values, which expressions are not specified. */
- public static final Object UNSPECIFIED_VALUE_PLACEHOLDER =
Placeholder.UNSPECIFIED_VALUE;
-
// We use enums for placeholders because enum
serialization/deserialization guarantees to preserve object's identity.
private enum Placeholder {
Review Comment:
This enum is no longer used. I think it can be safely removed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
