Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • E ensembl
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ensembl-gh-mirror
  • ensembl
  • Merge requests
  • !510

Merged
Created Aug 26, 2020 by Marek Szuba@mks

ENSCORESW-3534: xref Database is also a DBConnection object

  • Overview 2
  • Commits 1
  • Changes 1

Created by: magaliruffier

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
  • Review the contributing guidelines for this repository; remember in particular:
    • do not modify code without testing for regression
    • provide simple unit tests to test the changes
    • if you change the schema you must patch the test databases as well, see Updating the schema
    • the PR must not fail unit testing

Description

Convert Database object to inherit from DBConnection

Use case

DBConnection has a large number of methods implemented to deal with database connection management. the Database object used in the xref pipeline is created from the DBConnection object by adding the ISA, the Database object also inherits all the methods from the DBConnection object

Benefits

The Database object can use methods like disconnect_if_idle

Possible Drawbacks

NA

Testing

Have you added/modified unit tests to test the changes? Tried running the ParseSource runnable by adding disconnect_if_idle onto the newly created Database object This fails with the master code (disconnect_if_idle not available) but works with this code change

If so, do the tests pass/fail?

Have you run the entire test suite and no regression was detected?

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: feature/xref_dbconnection