You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
According to the dataflow documentation two properties are supported configuration-properties.classes and configuration-properties.names in the dataflow-configuration-metadata.properties file. This buildpack support only the first one - classes, completely ignoring names.
It would be nice to be able to specify also names.
Possible Solution
There is a question which is not quite clear to me. From what I can see in the documentation names and classes are additive. If my understanding is correct then the solution is as follows:
In boot/congifuration_metadata.go in NewDataFlowConfigurationMetadata function add a section to process names. It will need to iterate through properties, hints, and groups and add matching properties to Properties, Hints, and Groups respectively.
Motivation
While looking for a class mapping corresponding property and adding the class itself to the configuration-properties.classes property is a viable workaround, this path is confusing and to my best knowledge undocumented. Adding this feature will save a lot of hair loss to future users.
The text was updated successfully, but these errors were encountered:
Describe the Enhancement
According to the dataflow documentation two properties are supported
configuration-properties.classes
andconfiguration-properties.names
in thedataflow-configuration-metadata.properties
file. This buildpack support only the first one - classes, completely ignoring names.It would be nice to be able to specify also names.
Possible Solution
There is a question which is not quite clear to me. From what I can see in the documentation
names
andclasses
are additive. If my understanding is correct then the solution is as follows:In
boot/congifuration_metadata.go
inNewDataFlowConfigurationMetadata
function add a section to process names. It will need to iterate through properties, hints, and groups and add matching properties to Properties, Hints, and Groups respectively.Motivation
While looking for a class mapping corresponding property and adding the class itself to the
configuration-properties.classes
property is a viable workaround, this path is confusing and to my best knowledge undocumented. Adding this feature will save a lot of hair loss to future users.The text was updated successfully, but these errors were encountered: