pentaho / mondrian

Mondrian is an Online Analytical Processing (OLAP) server that enables business users to analyze large quantities of data in real-time.
http://mondrian.pentaho.com/
Other
1.14k stars 724 forks source link

fix: prevent AggColumnNameType from being directly used #1423

Closed hugo-jose closed 3 weeks ago

hugo-jose commented 3 weeks ago

Addressed Issue(s)

https://hv-eng.atlassian.net/browse/SME-313

Description

Having base types referenced at elements, introduces several serialization issues at jaxb generated code (at least at Semantic Model Editor project). This approach don't change the XSD structure or functionality, introducing two new types specific for those elements

Checklist

hitachivantarasonarqube[bot] commented 3 weeks ago

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

buildguy commented 3 weeks ago
[![👍 Frogbot scanned this pull request and did not find any new security issues.](https://raw.githubusercontent.com/jfrog/frogbot/master/resources/v2/noVulnerabilityBannerPR.png)](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)
Note: ---
**Frogbot** also supports **Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning**. This features are included as part of the [JFrog Advanced Security](https://jfrog.com/advanced-security) package, which isn't enabled on your system.

[🐸 JFrog Frogbot](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)
buildguy commented 3 weeks ago

:white_check_mark: Build finished in 17m 22s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -DrunITs -pl mondrian

:ok_hand: All tests passed!

Tests run: 3307, Failures: 0, Skipped: 0    Test Results


:information_source: This is an automatic message

graimundo commented 3 weeks ago

Missing Jira issue in commit message

hugo-jose commented 3 weeks ago

@graimundo Fixed

hugo-jose commented 3 weeks ago

SME code generation approach was modified and no longer requires this approach. PR will be closed.