DanielLeens commented on PR #10294: URL: https://github.com/apache/seatunnel/pull/10294#issuecomment-5083457803
Hi @LiJie20190102, thank you for bringing forward the StarRocks warehouse use case. Supporting warehouse-aware Stream Load is useful, and the PR helped us verify how this parameter should flow through the connector. On the current `dev` branch, the existing `starrocks.config` map is already forwarded to Stream Load headers, including `warehouse`. Adding a second `warehouse_name` option would therefore duplicate an existing path and introduce unclear precedence between the two configurations. We are closing this implementation because the runtime capability is already available through the generic configuration map. The use case is still valuable, and a small documentation-focused PR showing `starrocks.config.warehouse` would be warmly welcomed. Thank you for helping make this gap in discoverability visible. -- 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]
