arjun4084346 commented on code in PR #3967:
URL: https://github.com/apache/gobblin/pull/3967#discussion_r1631879233
##########
gobblin-runtime/src/main/java/org/apache/gobblin/runtime/api/FlowSpec.java:
##########
@@ -146,14 +150,13 @@ public static FlowSpec.Builder builder(URI catalogURI,
Properties flowProps) {
* @param key
* @param value
*/
- public synchronized void addProperty(String key, String value) {
+ public synchronized void addProperty(String key, Object value) {
Review Comment:
We should really remove configAsProperties
https://github.com/apache/gobblin/blob/master/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/api/FlowSpec.java#L95
or change configAsProperties to executionLevelProperties which are updated,
while the core configs which remain the same remain in `config`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
