You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2020/03/04 10:00:21 UTC
[GitHub] [incubator-doris] morningman opened a new pull request #3033:
[MemLimit] Normalize the setting of mem limit
morningman opened a new pull request #3033: [MemLimit] Normalize the setting of mem limit
URL: https://github.com/apache/incubator-doris/pull/3033
Normalize the setting of mem limit to avoid some unexpected exception.
For example, use may not setting query mem limit in query plan, which
may cause BE crash.
ISSUE: #3032
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] morningman merged pull request #3033: [MemLimit]
Normalize the setting of mem limit
Posted by GitBox <gi...@apache.org>.
morningman merged pull request #3033: [MemLimit] Normalize the setting of mem limit
URL: https://github.com/apache/incubator-doris/pull/3033
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] morningman commented on issue #3033: [MemLimit]
Normalize the setting of mem limit
Posted by GitBox <gi...@apache.org>.
morningman commented on issue #3033: [MemLimit] Normalize the setting of mem limit
URL: https://github.com/apache/incubator-doris/pull/3033#issuecomment-594501078
This is kind of self-defense. In case that when the caller forget to set the query mem limit, the BE will not crash.
`query mem limit` field in query plan structure is an optional fields, and user may forget to set it when building the query plan structure.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] chaoyli commented on issue #3033: [MemLimit]
Normalize the setting of mem limit
Posted by GitBox <gi...@apache.org>.
chaoyli commented on issue #3033: [MemLimit] Normalize the setting of mem limit
URL: https://github.com/apache/incubator-doris/pull/3033#issuecomment-594453315
why not set, will be trash?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org