-
Notifications
You must be signed in to change notification settings - Fork 0
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
Lb4/datahub rest emitter init #1
Conversation
…sions/control-plane/management-api/dataspace-catalog-api
...t-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/DatasetEntityIngestor.java
Dismissed
Show dismissed
Hide dismissed
...t-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/DatasetEntityIngestor.java
Dismissed
Show dismissed
Hide dismissed
...t-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/DatasetEntityIngestor.java
Fixed
Show fixed
Hide fixed
...nt-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/DomainEntityIngestor.java
Dismissed
Show dismissed
Hide dismissed
...nt-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/RelationshipIngestor.java
Dismissed
Show dismissed
Hide dismissed
...ment-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/UserEntityIngestor.java
Dismissed
Show dismissed
Hide dismissed
...nt-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/PolicyEntityIngestor.java
Dismissed
Show dismissed
Hide dismissed
...t-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/DatasetEntityIngestor.java
Fixed
Show fixed
Hide fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are always happy to welcome new contributors ❤️ To make things easier for everyone, please make sure to follow our contribution guidelines, check if you have already signed the ECA, and relate this pull request to an existing issue or discussion.
Test Results 555 files - 22 555 suites - 22 22m 42s ⏱️ -40s For more details on these failures, see this check. Results for commit bc2585b. ± Comparison against base commit 36f5a71. This pull request removes 167 and adds 95 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
…ec-int/edc into lb4/datahub_rest_emitter_init
...t-api/dataspace-catalog-api/src/main/java/be/imec/edit/ds/catalog/DatasetEntityIngestor.java
Fixed
Show fixed
Hide fixed
|
||
//set and emit data schema aspect to metadata model of datahub | ||
Future<MetadataWriteResponse> schemaPropsFut = emitter.emit(_metadataChangeProposalWrapper(_schemaMetadata(asset), entityType, datasetUrn)); | ||
var a = _editableDatasetProperties(asset); |
Check notice
Code scanning / CodeQL
Unread local variable Note
* */ | ||
|
||
public class DatasetEntityIngestor extends DataSpaceCatalogIngestorBase { | ||
Logger log = LoggerFactory.getLogger(this.getClass().getName()); |
Check warning
Code scanning / CodeQL
Field masks field in super class Warning
log
This pull request is stale because it has been open for 7 days with no activity. |
Add datahub emitter to the edc connector