Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MariaDB dialect detection #1090

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

petrprochy
Copy link

@petrprochy petrprochy commented Dec 14, 2018

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

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

@wingman-pentaho
Copy link
Collaborator

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
Copy link
Collaborator

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
Copy link
Member

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.

@lucboudreau lucboudreau added the jira Bug is entered in Jira's maintenance backlog label Dec 18, 2018
@petrprochy
Copy link
Author

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

Thanks and Marry Christmas :-)

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@buildguy

This comment has been minimized.

@hitachivantarasonarqube
Copy link

SonarQube Quality Gate:
Quality Gate failed

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

See analysis details on SonarQube

@buildguy

This comment has been minimized.

@buildguy
Copy link
Collaborator

❌ 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

@graimundo graimundo requested a review from a team as a code owner September 25, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jira Bug is entered in Jira's maintenance backlog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants