[GitHub] [cassandra] blerer commented on a diff in pull request #2193: CASSANDRA-17940: CEP-20: Dynamic Data Masking

["blerer (via GitHub)" <gi...@apache.org>]

blerer commented on code in PR #2193:
URL: https://github.com/apache/cassandra/pull/2193#discussion_r1146178294


##########
src/java/org/apache/cassandra/cql3/functions/masking/ColumnMask.java:
##########
@@ -93,6 +94,14 @@ public List<AbstractType<?>> partialArgumentTypes()
                : argTypes.subList(1, argTypes.size());
     }
 
+    /**
+     * @return The values of the arguments of the partially applied masking function.
+     */
+    public List<ByteBuffer> partialArgumentValues()
+    {
+        return Arrays.asList(partialArgumentValues);

Review Comment:
   We probably want to protect that List against modifications.



-- 
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: pr-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org