morningman commented on PR #20064:
URL: https://github.com/apache/doris/pull/20064#issuecomment-1565517379
The name `return_load_in_sink` is related to internal implementation, not
suitable for a user-oriented config' name.
There are 2 issues you want to solve in #19259
1. Test load performance
I think it is better to use a session variable or load properties for
this case. You can see there a session variable `dry_run_query`, when set to
true, it will run the query without return the result. It resolve the similar
issue as you want to resolve.
2. For materialized view.
This is indeed a table property. I suggest to think about new property
name instead of exposing the internal implements.
Eg: `"storage_policy" = "no_storage"`. (There is already a
"storage_policy" used for cold-hot data separation, maybe we can expend it)
All in all, my opinion is that the 2 things you want to resolve are 2
different cases. Better design it carefully to avoid adding confusing
properties.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
