Skip to content
This repository was archived by the owner on Oct 13, 2024. It is now read-only.

SCB-344 add document for how to use mysql as alpha's backend database#137

Merged
WillemJiang merged 6 commits into
apache:masterfrom
eric-lee-ltk:SCB-344
Feb 13, 2018
Merged

SCB-344 add document for how to use mysql as alpha's backend database#137
WillemJiang merged 6 commits into
apache:masterfrom
eric-lee-ltk:SCB-344

Conversation

@eric-lee-ltk
Copy link
Copy Markdown
Contributor

@eric-lee-ltk eric-lee-ltk commented Feb 13, 2018

Follow this checklist to help us incorporate your contribution quickly and easily:

  • Make sure there is a JIRA issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a JIRA issue. Your pull request should address just this issue, without pulling in other changes.
  • Each commit in the pull request should have a meaningful subject line and body.
  • Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

This PR is based on #135 .

As the default table name in eclipselink is upper case and table name in query is case sensitive, we need to explicitly tell the query the exact table name. That's why the @Table came for.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 95.273% when pulling d124955 on eric-lee-ltk:SCB-344 into 697bf60 on apache:master.

@WillemJiang WillemJiang merged commit 5e37786 into apache:master Feb 13, 2018
@eric-lee-ltk eric-lee-ltk deleted the SCB-344 branch February 13, 2018 08:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants