Skip to content

Commit

Permalink
Merge branch 'main' into p1_validate-asset-policy-cd
Browse files Browse the repository at this point in the history
  • Loading branch information
kamilczaja authored Jul 18, 2024
2 parents 8f30c7d + 1687477 commit e824699
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
12 changes: 9 additions & 3 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,12 @@ please see [changelog_updates.md](docs/dev/changelog_updates.md).

### Deployment Migration Notes

New configuration to access the database:

- `EDC_SERVER_DB_CONNECTION_POOL_SIZE`
- The property controls the maximum size that the pool is allowed to reach, including both idle and in-use connections. Basically this value will determine the maximum number of actual connections to the database backend.
- Defaults to `3`

#### Compatible Versions

- Connector Backend Docker Images:
Expand All @@ -42,7 +48,7 @@ MDS 2.2 intermediate release

- API Wrapper UI API: Data sources are now well-typed.
- The Broker has been removed in favor of the Authority Portal:
- A new Deployment Unit, the ["Data Catalog Crawler"](extensions/catalog-crawler/README.md), has been added.
- A new Deployment Unit, the ["Data Catalog Crawler"](extensions/catalog-crawler/README.md), has been added.
- Each "Data Catalog Crawler" connects to an existing Authority Portal Deployment's DB.
- Each "Data Catalog Crawler" is responsible for crawling exactly one environment.
- The Data Catalog functionality of the Broker has been integrated into the Authority Portal.
Expand All @@ -64,8 +70,8 @@ MDS 2.2 intermediate release
- Connector:
- The database migration system has been moved from multiple migration history tables to a single one.
- Broker:
- The broker has been removed. For Authority Portal users, please check out the new
[Data Catalog Crawler Productive Deployment Guide](docs/deployment-guide/goals/catalog-crawler-production/README.md).
- The broker has been removed. For Authority Portal users, please check out the new
[Data Catalog Crawler Productive Deployment Guide](docs/deployment-guide/goals/catalog-crawler-production/README.md).
- Any previous broker deployment's database is not required anymore.
- Please care that only some environment variables look similar. It is recommended to create fresh deployments.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,6 @@ public class DatabaseDirectAccessExtension implements ServiceExtension {
@Setting(defaultValue = "3")
public static final String EDC_SERVER_DB_CONNECTION_POOL_SIZE = "edc.server.db.connection.pool.size";

/**
* Sets the connection timeout for the datasource in milliseconds.
*/
@Setting(defaultValue = "5000")
public static final String EDC_SERVER_DB_CONNECTION_TIMEOUT_IN_MS = "edc.server.db.connection.timeout.in.ms";


@Override
public String name() {
Expand All @@ -75,11 +69,11 @@ private void initializeDirectDatabaseAccess(ServiceExtensionContext context) {
hikariConfig.setUsername(config.getString(EDC_DATASOURCE_JDBC_USER));
hikariConfig.setPassword(config.getString(EDC_DATASOURCE_JDBC_PASSWORD));
hikariConfig.setMinimumIdle(1);
hikariConfig.setMaximumPoolSize(config.getInteger(EDC_SERVER_DB_CONNECTION_POOL_SIZE));
hikariConfig.setMaximumPoolSize(config.getInteger(EDC_SERVER_DB_CONNECTION_POOL_SIZE, 3));
hikariConfig.setIdleTimeout(30000);
hikariConfig.setPoolName("direct-database-access");
hikariConfig.setMaxLifetime(50000);
hikariConfig.setConnectionTimeout(config.getInteger(EDC_SERVER_DB_CONNECTION_TIMEOUT_IN_MS));
hikariConfig.setConnectionTimeout(1000);

val dda = new DslContextFactory(new HikariDataSource(hikariConfig));

Expand Down

0 comments on commit e824699

Please sign in to comment.