MariaDB dialect detection

[petrprochy]

Improved detection of MariaDB database and fixed behaviour of method allowFromQueue().

Link to JIRA: https://jira.pentaho.com/browse/MONDRIAN-2650

[wingman-pentaho]

Hello, This is an automated comment from our build system. Since this pull request originated from a user not in the Pentaho organization, we will not proceed with automatic validation of the pull request. However, rest assured that it will be reviewed manually as soon as we get the chance. Thanks for your contribution!

[wingman-pentaho]

Build Completed

❌ This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.

Build Commands

mvn -B -f 'pom.xml' -pl 'mondrian' -P '!assemblies' -DrunITs=true -Dsurefire.runOrder=alphabetical -Dfailsafe.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands

mvn -B -f 'pom.xml' -pl 'mondrian' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files

  mondrian/src/it/java/mondrian/spi/impl/MariaDBDialectTest.java
  mondrian/src/main/java/mondrian/spi/impl/JdbcDialectImpl.java
  mondrian/src/main/java/mondrian/spi/impl/MariaDBDialect.java

Integration Test Coverage

These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

⚠️ Coverage Changes (click to expand)

mondrian.olap.Util.Flat3List

• Branch Change: -2.7% 🔻
• Complexity Change: -3.2% 🔻

mondrian.rolap.agg.SegmentCacheManager.AbortException

• Complexity Change: -100% 🔻
• Instruction Change: -100% 🔻
• Line Change: -100% 🔻
• Method Change: -100% 🔻

mondrian.rolap.agg.SegmentCacheManager.Actor

• Branch Change: -25% 🔻
• Complexity Change: -12.5% 🔻
• Instruction Change: -8.6% 🔻
• Line Change: -8.8% 🔻

mondrian.rolap.agg.SegmentLoader

• Branch Change: -1.5% 🔻
• Complexity Change: -2.4% 🔻
• Instruction Change: -0.7% 🔻
• Line Change: -0.8% 🔻

mondrian.rolap.agg.SegmentLoader.2.new SegmentCacheManager.Command() {...}

• Branch Change: -33.3% 🔻
• Complexity Change: -33.3% 🔻
• Instruction Change: -6.8% 🔻
• Line Change: -6.7% 🔻

mondrian.rolap.cache.SegmentCacheIndexImpl

• Branch Change: -1.4% 🔻
• Complexity Change: -2.2% 🔻
• Instruction Change: -1.2% 🔻
• Line Change: -1.5% 🔻

mondrian.rolap.FastBatchingCellReader

• Branch Change: -1.9% 🔻
• Complexity Change: -3.1% 🔻
• Instruction Change: -0.4% 🔻
• Line Change: -1.1% 🔻

mondrian.server.MonitorImpl.Handler

• Branch Change: -2.2% 🔻
• Complexity Change: -1.8% 🔻
• Instruction Change: -0.5% 🔻
• Line Change: -0.5% 🔻

mondrian.spi.impl.JdbcDialectImpl

• Complexity Change: -0.3% 🔻

mondrian.spi.impl.MariaDBDialect

• Complexity Change: -5% 🔻
• Instruction Change: -2.6% 🔻
• Line Change: -1.8% 🔻
• Method Change: -8.3% 🔻

License header violations

❗ Couldn't find acceptable copyright on mondrian/src/it/java/mondrian/spi/impl/MariaDBDialectTest.java

❗ Couldn't find acceptable license on mondrian/src/it/java/mondrian/spi/impl/MariaDBDialectTest.java

❗ Copyright year is not the current year on mondrian/src/main/java/mondrian/spi/impl/MariaDBDialect.java. Found 2017. Was expecting 2018

[lucboudreau]

This is a good start, but the PR is incomplete. We will have to add some missing parts when we get to integrating it. HV folks: See the Jira comments for details.

[petrprochy]

Thanks for your reply Luc,
I committed changes which failed build.
Please, could you review changes?

Thanks and Marry Christmas :-)

[hitachivantarasonarqube]

SonarQube Quality Gate:

Maintainability Rating on New Code (is worse than A)
3 New Code Smells (is greater than 0)

See analysis details on SonarQube

[buildguy]

❌ Build failed in 1h 45m 44s

Build command:

mvn clean verify -B -e -Daudit -amd -DrunITs -pl mondrian

---

❗ No tests found!

Errors:

Filtered log (click to expand)

script returned exit code 1

ℹ️ This is an automatic message