github-advanced-security[bot] commented on code in PR #15633:
URL: https://github.com/apache/druid/pull/15633#discussion_r1443648607


##########
processing/src/main/java/org/apache/druid/segment/VirtualColumn.java:
##########
@@ -299,13 +300,16 @@
 
   /**
    * Get the {@link ColumnIndexSupplier} for the specified virtual column, 
with the assistance of a
-   * {@link ColumnSelector} to allow reading things from segments. If the 
virtual column has no indexes, this method
-   * will return null, or may also return a non-null supplier whose methods 
may return null values - having a supplier
-   * is no guarantee that the column has indexes.
+   * {@link ColumnIndexSelector} to allow reading things from segments. If the 
virtual column has no indexes, this
+   * method will return null, or may also return a non-null supplier whose 
methods may return null values - having a
+   * supplier is no guarantee that the column has indexes.
    */
   @SuppressWarnings("unused")
   @Nullable
-  default ColumnIndexSupplier getIndexSupplier(String columnName, 
ColumnSelector columnSelector)
+  default ColumnIndexSupplier getIndexSupplier(
+      String columnName,

Review Comment:
   ## Useless parameter
   
   The parameter 'columnName' is never used.
   
   [Show more 
details](https://github.com/apache/druid/security/code-scanning/6422)



-- 
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]

Reply via email to