fix: replace string simple type usages with a complex type [SME-313]

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: replace string simple type usages with a complex type [SME-313] #1424

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 simple types referenced at elements, introduces issues at jaxb generated code at Semantic Model Editor project. SME requires the injection of a new smeID atribute at the generated types, that is not possible with the primitive String type.

Checklist

[ ] Relevant tests have been added
[ ] This PR should not be squashed

hitachivantarasonarqube[bot] commented 3 weeks ago

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 31s

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