Skip to content

Commit c1bc066

Browse files
attilapirospan3793
authored andcommitted
[KYUUBI #7299] [AUTHZ] Remove redundant concatenation with the list of conditions
### Why are the changes needed? This concatenation with the list of conditions `conditionList` is redundant as the `columnPrune` method already contains it. ### How was this patch tested? Existing unit tests. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #7299 from attilapiros/apiros_fix_1. Closes #7299 18af358 [attilapiros] Remove redundant concatenation with the list of conditions Authored-by: attilapiros <[email protected]> Signed-off-by: Cheng Pan <[email protected]> (cherry picked from commit 526a8aa) Signed-off-by: Cheng Pan <[email protected]>
1 parent 394201b commit c1bc066

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilder.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ object PrivilegesBuilder {
6969
if (projectionList.isEmpty) {
7070
privilegeObjects += PrivilegeObject(table, plan.output.map(_.name))
7171
} else {
72-
val cols = columnPrune(projectionList ++ conditionList, plan.outputSet)
72+
val cols = columnPrune(projectionList, plan.outputSet)
7373
privilegeObjects += PrivilegeObject(table, cols.map(_.name).distinct)
7474
}
7575
}

0 commit comments

Comments
 (0)