Skip to content

Commit 0383507

Browse files
optimize: reduce a partitions call when checking permissions (#376)
## Summary ## Checklist - [ ] Added Unit Tests - [ ] Covered by existing CI - [ ] Integration tested - [ ] Documentation update <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Modified join analysis behavior to disable automatic table permission checks by default, simplifying operations. Users can now explicitly enable permission validation when needed. <!-- end of auto-generated comment: release notes by coderabbit.ai --> <!-- av pr metadata This information is embedded by the av CLI when creating PRs to track the status of stacks when using Aviator. Please do not delete or edit this section of the PR. ``` {"parent":"main","parentHead":"","trunk":"main"} ``` --> Co-authored-by: Thomas Chow <[email protected]>
1 parent 7294e96 commit 0383507

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

spark/src/main/scala/ai/chronon/spark/Analyzer.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,7 @@ class Analyzer(tableUtils: TableUtils,
273273

274274
def analyzeJoin(joinConf: api.Join,
275275
skewDetection: Boolean = false,
276-
validateTablePermission: Boolean = true,
276+
validateTablePermission: Boolean = false,
277277
validationAssert: Boolean = false): (Map[String, DataType], ListBuffer[AggregationMetadata]) = {
278278
val name = "joins/" + joinConf.metaData.name
279279
logger.info(s"""|Running join analysis for $name ...\n""".stripMargin)

0 commit comments

Comments
 (0)