[GitHub] [incubator-seatunnel] RickyHuo commented on issue #904: [Feature][config] improve the config parse logic

[GitBox <gi...@apache.org>]

RickyHuo commented on issue #904:
URL: https://github.com/apache/incubator-seatunnel/issues/904#issuecomment-1002949926


   I have saw the draft PR. The seatunnel-config has been replaced with typesafe config.
   
   The reason why we develop seatunnel-config base on typesafe config is that the plugin order of parsed Config is not the same as our configuration when transform section has multiply plugins.
   
   The origin issue is #29, I'm sorry I forget config test at that moment.
   
   We should test those demo:
   
   ```
   source {
   	
   }
   
   transform {
   	
   	sql {
   		sql = "sql1"
   	}
   
   	sql {
   		sql = "sql2"
   	}
   
   	sql {
   		sql = "sql3"
   	}
   }
   
   sink {
   	
   }
   
   ```
   
   
   ```
   source {
   	
   }
   
   transform {
   	
   	sql {
   		sql = "sql1"
   	}
   
   	json {
   		source_field = "xxx"
   	}
   
   	sql {
   		sql = "sql3"
   	}
   }
   
   sink {
   	
   }
   
   ```
   


-- 
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: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org