[GitHub] [incubator-seatunnel] ruanwenjun commented on issue #1829: [Feature][flink-connector-jdbc] JdbcSink query sql supports param placeholder replacing row index.

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

ruanwenjun commented on issue #1829:
URL: https://github.com/apache/incubator-seatunnel/issues/1829#issuecomment-1123110254

   >1: `insert into test(name,age) values(#{name},#{age}) ` is more brief semantics.
   I don't think so, I think `insert into test(name,age) values(?,?)` is more easy and is standard SQL.
   >2: jdbc upsert support
   This is interesting, but I would want to know this works, I guess it will only be able to upsert in the current jdbc batch?
   


-- 
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