cecemei commented on code in PR #17055:
URL: https://github.com/apache/druid/pull/17055#discussion_r1763822659
##########
processing/src/main/java/org/apache/druid/query/filter/Filter.java:
##########
@@ -37,6 +38,25 @@
@SubclassesMustOverrideEqualsAndHashCode
public interface Filter
{
+ default String getFilterString()
+ {
+ return toString();
+ }
+
+ /**
+ * Returns an ordered set of all child filters for this filter with no
duplicates. The default return value is an
+ * empty set.
+ * <p>
+ * Only {@link org.apache.druid.segment.filter.AndFilter} and {@link
org.apache.druid.segment.filter.OrFilter} have
+ * child filters. Other filters would return empty (including {@link
org.apache.druid.segment.filter.NotFilter}).
+ * <p>
+ * The ordering of child filters is important in some cases,
e.x.short-curcuiting.
+ */
+ default ImmutableSet<Filter> getFilters()
Review Comment:
Ok reverted this change
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
