Skip to content

Conversation

@Shreyansh228
Copy link

Within the context of the dagger post-processor, when enhancing data through an external API, the response is presently linked to an extra field within the output-proto. However, if the intention is to associate it with an existing nested field instead, the current approach falls short. This adjustment aims to enable the mapping of this response to the pre-existing nested field within the output proto. By doing so, the need for introducing the additional field will be avoided, resulting in a more efficient proto schema mapping for the output proto.

e.g
Mapping of external API response to the nested existing field driver-pickup-location.address in the output proto here
By this way we can avoid introducing new field address in the output proto.

"output_mapping": {
    "driver-pickup-location.address": {
	    "path": "$.driver_pickup_location.address"
    }
}

Shreyansh added 2 commits August 29, 2023 13:34
In order to appropriately organize the proto, ensuring the mapping of external API responses (whether gRPC or HTTP) to the nested fields in the output proto is efficient. Additionally, this approach, while maintaining compatibility with the prior method of associating external responses to predefined fields, avoids the need to introduce supplementary fields in the output proto.
@lavkesh lavkesh merged commit 4df7081 into main Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants