Skip to content

Conversation

devops-unitybars
Copy link
Contributor

Issue #197
This pull request introduces enhanced parsing support for the PIVOT and UNPIVOT clauses within PL/SQL statements in the ZPA plugin. The implementation adheres to the syntax specifications detailed in the official Oracle documentation (https://docs.oracle.com/html/E26088_02/statements_10002.htm#CHDCEJJE). These enhancements improve the plugin's ability to accurately parse and analyze queries that utilize these advanced SQL constructs, thereby increasing compatibility with contemporary Oracle SQL features.

@felipebz felipebz merged commit 4315250 into felipebz:main Jun 26, 2025
felipebz pushed a commit that referenced this pull request Jun 26, 2025
* fix parrsing 'pivot' and 'unpivot' constructs

* Merge branch 'felipebz:main' into fix-parsing-pivot-and-unpivot-const…

* add pivot/unpivot test

* add pivot/unpivot test
felipebz pushed a commit that referenced this pull request Jun 26, 2025
* fix parrsing 'pivot' and 'unpivot' constructs

* Merge branch 'felipebz:main' into fix-parsing-pivot-and-unpivot-const…

* add pivot/unpivot test

* add pivot/unpivot test
felipebz pushed a commit that referenced this pull request Jun 26, 2025
* fix parrsing 'pivot' and 'unpivot' constructs

* Merge branch 'felipebz:main' into fix-parsing-pivot-and-unpivot-const…

* add pivot/unpivot test

* add pivot/unpivot test
@felipebz
Copy link
Owner

Hi,

Thanks a lot for your contribution!

I accidentally merged the PR before leaving any comments here 😅. Just a heads-up that I made a couple of small adjustments after the merge:

  • Reformatted the JSON files under zpa-checks/src/integrationTest/resources/expected/(...) to keep them consistent with the original style. That helps highlight the actual diffs.
  • Removed an unrelated SQL snippet that had been added to insert_without_columns.sql.

Again, really appreciate the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants