Skip to content

Commit

Permalink
[SPARK-48114][SQL] Move subquery validation out of CheckAnalysis
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

Move subquery validation out of `CheckAnalysis`.

### Why are the changes needed?

To be reused in the single-pass Analyzer.

### Does this PR introduce _any_ user-facing change?

No, refactoring.

### How was this patch tested?

Existing tests.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #49994 from vladimirg-db/vladimirg-db/move-subquery-validation-out-of-check-analysis.

Authored-by: Vladimir Golubev <vladimir.golubev@databricks.com>
Signed-off-by: Max Gekk <max.gekk@gmail.com>
(cherry picked from commit ead7d58)
Signed-off-by: Max Gekk <max.gekk@gmail.com>
  • Loading branch information
vladimirg-db authored and MaxGekk committed Feb 18, 2025
1 parent ebc8df2 commit 0b627ae
Show file tree
Hide file tree
Showing 4 changed files with 578 additions and 520 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ object AnalysisContext {
* [[UnresolvedRelation]]s into fully typed objects using information in a [[SessionCatalog]].
*/
class Analyzer(override val catalogManager: CatalogManager) extends RuleExecutor[LogicalPlan]
with CheckAnalysis with SQLConfHelper with ColumnResolutionHelper {
with CheckAnalysis with AliasHelper with SQLConfHelper with ColumnResolutionHelper {

private val v1SessionCatalog: SessionCatalog = catalogManager.v1SessionCatalog
private val relationResolution = new RelationResolution(catalogManager)
Expand Down
Loading

0 comments on commit 0b627ae

Please sign in to comment.