Skip to content

Commit a10cc61

Browse files
committed
[SC-5451] Fix databricks/spark branch-2.1 build because SortPartitions is removed
## What changes were proposed in this pull request? Fix databricks/spark branch-2.1 build because SortPartitions is removed ## How was this patch tested? jenkins Author: Yin Huai <yhuai@databricks.com> Closes apache#150 from yhuai/branch-2.1-fix.
1 parent f39db8f commit a10cc61

File tree

2 files changed

+0
-3
lines changed

2 files changed

+0
-3
lines changed

sql/core/src/main/scala/com/databricks/sql/acl/CheckPermissions.scala

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,6 @@ class CheckPermissions(catalog: PublicCatalog, aclClient: AclClient)
240240
case _: SerializeFromObject => true
241241
case _: Aggregate => true
242242
case _: RepartitionByExpression => true
243-
case _: SortPartitions => true
244243
case _: SubqueryAlias => true
245244
case _: Expand => true
246245
case _: LocalLimit => true

sql/core/src/test/scala/com/databricks/sql/acl/CheckPermissionRuleSuite.scala

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,8 +163,6 @@ class CheckPermissionRuleSuite extends CheckRequests {
163163
subqueryExpr, anonymousExpr),
164164
add("RepartitionByExpression", plan => RepartitionByExpression(Seq(subqueryExpr), plan),
165165
subqueryExpr),
166-
add("SortPartition", plan => SortPartitions(Seq(SortOrder(subqueryExpr, Ascending)), plan),
167-
subqueryExpr),
168166
add("SubqueryAlias", plan => SubqueryAlias("a", plan, None)),
169167
add("Expand", plan => Expand(Seq.empty, Seq.empty, plan)),
170168
add("Sort", plan => Sort(Seq(SortOrder(subqueryExpr, Ascending)), true, plan), subqueryExpr),

0 commit comments

Comments
 (0)