[ 
https://issues.apache.org/jira/browse/IGNITE-7195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yakov Zhdanov updated IGNITE-7195:
----------------------------------
    Description: 
GridToStringBuilder should limit large collections output to first N elements 
(otherwise, OOME is possible). N should be limited by 
IGNITE_TO_STRING_COLLECTION_LIMIT system property and should be 10 by default.

Upd: how about we limit GridToStringBuilder to 10000 chars by default to avoid 
OOME in complex recursive obj graphs? and (1) provide system property to change 
default and (2) also provide S.toString() overload that will accept maxLen 
parameter. 

Upd1: how about we add thread local message length counter in addition to 
thread local with buffers.

  was:
GridToStringBuilder should limit large collections output to first N elements 
(otherwise, OOME is possible). N should be limited by 
IGNITE_TO_STRING_COLLECTION_LIMIT system property and should be 10 by default.

Upd: how about we limit GridToStringBuilder to 10000 chars by default to avoid 
OOME in complex recursive obj graphs? and (1) provide system property to change 
default and (2) also provide S.toString() overload that will accept maxLen 
parameter. 


> GridToStringBuilder should limit large collections output to first N elements
> -----------------------------------------------------------------------------
>
>                 Key: IGNITE-7195
>                 URL: https://issues.apache.org/jira/browse/IGNITE-7195
>             Project: Ignite
>          Issue Type: Bug
>            Reporter: Yakov Zhdanov
>
> GridToStringBuilder should limit large collections output to first N elements 
> (otherwise, OOME is possible). N should be limited by 
> IGNITE_TO_STRING_COLLECTION_LIMIT system property and should be 10 by default.
> Upd: how about we limit GridToStringBuilder to 10000 chars by default to 
> avoid OOME in complex recursive obj graphs? and (1) provide system property 
> to change default and (2) also provide S.toString() overload that will accept 
> maxLen parameter. 
> Upd1: how about we add thread local message length counter in addition to 
> thread local with buffers.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to