Skip to content

correction in case of empty dataset service payload for manifest by reference

Devdatta Santra requested to merge Empty_Data_Payload_Fix_for_Manifest_By_Ref into master

Type of change

  • Bug Fix
  • Feature

Does this introduce a change in the core logic?

  • [Yes]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Updates description?

This is related to issue -osdu/platform/data-flow/ingestion/ingestion-dags#108 (closed)

The error encountered while running the 'manifest by reference' DAG - AttributeError: 'dict' object has no attribute 'to_JSON'

The _post_instructions method expects the argument data to be GetDatasetRegistryRequest. However, the method is called with empty dict. So, an extra check is added to handle such situation, when the data field is not passed to the request. see also - osdu/platform/data-flow/ingestion/ingestion-dags#108 (comment 159368)

Merge request reports