[DISCUSSION] Urgent Ignite bug fix release

classic Classic list List threaded Threaded
41 messages Options
123
Reply | Threaded
Open this post in threaded view
|

[DISCUSSION] Urgent Ignite bug fix release

dmagda
Igniters,

Presently every Apache Ignite node requests 80% of RAM from an operating system. We debated a lot about this default [1] and as a practice shows those who voted for the lower value (20%-25%) were right.

Our users started reporting about bizarre issues ([2], [3], [4]) when a cluster hangs, freezes for a while or even crashes during standard preloading phase or minor workloads.

It’s turned out to be that lack of RAM causes extensive swapping and checkpointing activity. Even more, the issue can be reproduced on your laptop if you launch a couple of nodes and start data preloading.

All this can be avoided if you allocated less than 80% and tune the checkpointing if the persistence is used. Unfortunately, only few could get to this point, the rest would kick Ignite out from their deployments.

Thus, we have to fix the defaults ([5] and [6]) on our own and release an emergency release with the fixes as soon as possible.

Any objections or other thoughts?

[1] http://apache-ignite-developers.2346864.n4.nabble.com/IGNITE-5717-improvements-of-MemoryPolicy-default-size-td20264.html
[2] http://apache-ignite-users.70518.x6.nabble.com/Performance-of-persistent-store-too-low-when-bulb-loading-tp16247p16337.html
[3] http://apache-ignite-users.70518.x6.nabble.com/Strange-problems-with-Ignite-native-Persistence-when-Data-exceeds-Memory-td16187.html#a16194
[4] http://apache-ignite-users.70518.x6.nabble.com/Activation-slow-and-Ignite-node-crashed-in-the-middle-of-checkpoint-td16144.html

[5] https://issues.apache.org/jira/browse/IGNITE-6003
[6] https://issues.apache.org/jira/browse/IGNITE-6182


Denis
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dsetrakyan
+1 on the release, let's do it ASAP.

On Thu, Aug 24, 2017 at 12:32 PM, Denis Magda <[hidden email]> wrote:

> Igniters,
>
> Presently every Apache Ignite node requests 80% of RAM from an operating
> system. We debated a lot about this default [1] and as a practice shows
> those who voted for the lower value (20%-25%) were right.
>
> Our users started reporting about bizarre issues ([2], [3], [4]) when a
> cluster hangs, freezes for a while or even crashes during standard
> preloading phase or minor workloads.
>
> It’s turned out to be that lack of RAM causes extensive swapping and
> checkpointing activity. Even more, the issue can be reproduced on your
> laptop if you launch a couple of nodes and start data preloading.
>
> All this can be avoided if you allocated less than 80% and tune the
> checkpointing if the persistence is used. Unfortunately, only few could get
> to this point, the rest would kick Ignite out from their deployments.
>
> Thus, we have to fix the defaults ([5] and [6]) on our own and release an
> emergency release with the fixes as soon as possible.
>
> Any objections or other thoughts?
>
> [1] http://apache-ignite-developers.2346864.n4.nabble.
> com/IGNITE-5717-improvements-of-MemoryPolicy-default-size-td20264.html
> [2] http://apache-ignite-users.70518.x6.nabble.com/
> Performance-of-persistent-store-too-low-when-bulb-
> loading-tp16247p16337.html
> [3] http://apache-ignite-users.70518.x6.nabble.com/Strange-
> problems-with-Ignite-native-Persistence-when-Data-exceeds-
> Memory-td16187.html#a16194
> [4] http://apache-ignite-users.70518.x6.nabble.com/
> Activation-slow-and-Ignite-node-crashed-in-the-middle-of-
> checkpoint-td16144.html
>
> [5] https://issues.apache.org/jira/browse/IGNITE-6003
> [6] https://issues.apache.org/jira/browse/IGNITE-6182
>
> —
> Denis
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

Ivan Rakov
Let's include to the release tickets with optimizations of checkpointing
algorithm:

https://issues.apache.org/jira/browse/IGNITE-6178
https://issues.apache.org/jira/browse/IGNITE-6033
https://issues.apache.org/jira/browse/IGNITE-5961

This will help users who are experiencing problems with slow checkpoints.

Also, let's include https://issues.apache.org/jira/browse/IGNITE-6183 to
soften "Ignite node crashed in the middle of checkpoint" message as per
discussion
<http://apache-ignite-developers.2346864.n4.nabble.com/Ignite-close-G-stop-name-true-Change-flag-cancel-to-false-td20473.html>.

Best Regards,
Ivan Rakov

On 25.08.2017 3:31, Dmitriy Setrakyan wrote:

> +1 on the release, let's do it ASAP.
>
> On Thu, Aug 24, 2017 at 12:32 PM, Denis Magda <[hidden email]> wrote:
>
>> Igniters,
>>
>> Presently every Apache Ignite node requests 80% of RAM from an operating
>> system. We debated a lot about this default [1] and as a practice shows
>> those who voted for the lower value (20%-25%) were right.
>>
>> Our users started reporting about bizarre issues ([2], [3], [4]) when a
>> cluster hangs, freezes for a while or even crashes during standard
>> preloading phase or minor workloads.
>>
>> It’s turned out to be that lack of RAM causes extensive swapping and
>> checkpointing activity. Even more, the issue can be reproduced on your
>> laptop if you launch a couple of nodes and start data preloading.
>>
>> All this can be avoided if you allocated less than 80% and tune the
>> checkpointing if the persistence is used. Unfortunately, only few could get
>> to this point, the rest would kick Ignite out from their deployments.
>>
>> Thus, we have to fix the defaults ([5] and [6]) on our own and release an
>> emergency release with the fixes as soon as possible.
>>
>> Any objections or other thoughts?
>>
>> [1] http://apache-ignite-developers.2346864.n4.nabble.
>> com/IGNITE-5717-improvements-of-MemoryPolicy-default-size-td20264.html
>> [2] http://apache-ignite-users.70518.x6.nabble.com/
>> Performance-of-persistent-store-too-low-when-bulb-
>> loading-tp16247p16337.html
>> [3] http://apache-ignite-users.70518.x6.nabble.com/Strange-
>> problems-with-Ignite-native-Persistence-when-Data-exceeds-
>> Memory-td16187.html#a16194
>> [4] http://apache-ignite-users.70518.x6.nabble.com/
>> Activation-slow-and-Ignite-node-crashed-in-the-middle-of-
>> checkpoint-td16144.html
>>
>> [5] https://issues.apache.org/jira/browse/IGNITE-6003
>> [6] https://issues.apache.org/jira/browse/IGNITE-6182
>>
>> —
>> Denis

Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

yzhdanov
Agree, let's release new version including tickets mentioned by Denis and
Ivan.

--Yakov
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

gvvinblade
What do you, guys think about next warning?

[2017-08-25 17:17:04,718][INFO
][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
System cache's MemoryPolicy size is configured to 40 MB. Use
MemoryConfiguration.systemCacheMemorySize property to change the setting.
[2017-08-25 17:17:04,718][WARN
][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]

>>> Required RAM size is larger than total physical memory available for OS.
>>> Check your configuration to avoid swap partition usage.
>>> Use MemoryConfiguration and MemoryPolicyConfiguration to change the
settings.
>>> Physical memory [required=16397MB, available=15942MB]

[2017-08-25 17:17:04,726][WARN
][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
Peer class loading is enabled (disable it in production for performance and
deployment consistency reasons)
[2017-08-25 17:17:04,726][INFO
][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
Configured caches [in 'sysMemPlc' memoryPolicy: ['ignite-sys-cache']]
[2017-08-25 17:17:04,731][INFO
][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
3-rd party licenses can be found at:
/home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses


2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:

> Agree, let's release new version including tickets mentioned by Denis and
> Ivan.
>
> --Yakov
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dsetrakyan
Igor, I would change the message. How about this:

Required RAM size is larger than total physical memory available for OS.
> Please change MemoryConfiguration.WhichProperty and
> MemoryPolicyConfiguration.WhichProperty to decrease memory allocated for
> each node.


Also, can we calculate what the memory size allocated for each node should
be? In that case we should suggest it.

D.


On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <[hidden email]>
wrote:

> What do you, guys think about next warning?
>
> [2017-08-25 17:17:04,718][INFO
> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> System cache's MemoryPolicy size is configured to 40 MB. Use
> MemoryConfiguration.systemCacheMemorySize property to change the setting.
> [2017-08-25 17:17:04,718][WARN
> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
>
> >>> Required RAM size is larger than total physical memory available for
> OS.
> >>> Check your configuration to avoid swap partition usage.
> >>> Use MemoryConfiguration and MemoryPolicyConfiguration to change the
> settings.
> >>> Physical memory [required=16397MB, available=15942MB]
>
> [2017-08-25 17:17:04,726][WARN
> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> Peer class loading is enabled (disable it in production for performance and
> deployment consistency reasons)
> [2017-08-25 17:17:04,726][INFO
> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> Configured caches [in 'sysMemPlc' memoryPolicy: ['ignite-sys-cache']]
> [2017-08-25 17:17:04,731][INFO
> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> 3-rd party licenses can be found at:
> /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
>
>
> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>
> > Agree, let's release new version including tickets mentioned by Denis and
> > Ivan.
> >
> > --Yakov
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

Sergey Kozlov
I suppose we should not forget JVM heap size and suggest to reduce both
options

On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <[hidden email]>
wrote:

> Igor, I would change the message. How about this:
>
> Required RAM size is larger than total physical memory available for OS.
> > Please change MemoryConfiguration.WhichProperty and
> > MemoryPolicyConfiguration.WhichProperty to decrease memory allocated for
> > each node.
>
>
> Also, can we calculate what the memory size allocated for each node should
> be? In that case we should suggest it.
>
> D.
>
>
> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <[hidden email]>
> wrote:
>
> > What do you, guys think about next warning?
> >
> > [2017-08-25 17:17:04,718][INFO
> > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> > System cache's MemoryPolicy size is configured to 40 MB. Use
> > MemoryConfiguration.systemCacheMemorySize property to change the
> setting.
> > [2017-08-25 17:17:04,718][WARN
> > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> >
> > >>> Required RAM size is larger than total physical memory available for
> > OS.
> > >>> Check your configuration to avoid swap partition usage.
> > >>> Use MemoryConfiguration and MemoryPolicyConfiguration to change the
> > settings.
> > >>> Physical memory [required=16397MB, available=15942MB]
> >
> > [2017-08-25 17:17:04,726][WARN
> > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> > Peer class loading is enabled (disable it in production for performance
> and
> > deployment consistency reasons)
> > [2017-08-25 17:17:04,726][INFO
> > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> > Configured caches [in 'sysMemPlc' memoryPolicy: ['ignite-sys-cache']]
> > [2017-08-25 17:17:04,731][INFO
> > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePathSelfTest0]
> > 3-rd party licenses can be found at:
> > /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
> >
> >
> > 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> >
> > > Agree, let's release new version including tickets mentioned by Denis
> and
> > > Ivan.
> > >
> > > --Yakov
> > >
> >
>



--
Sergey Kozlov
GridGain Systems
www.gridgain.com
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

gvvinblade
An example of current impl:


[2017-08-25 19:44:37,740][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
[2017-08-25 19:44:37,740][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
Not enough memory for current process [required=29251MB, available=15942MB].
[2017-08-25 19:44:37,740][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
Please change MemoryConfiguration.systemCacheMaxSize and
MemoryConfiguration.defaultMemoryPolicySize to decrease memory allocated
for each node.
[2017-08-25 19:44:37,740][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
[2017-08-25 19:44:37,740][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
Current settings:
[2017-08-25 19:44:37,740][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
  HeapInit=250MB
[2017-08-25 19:44:37,741][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
  HeapMax=3543MB
[2017-08-25 19:44:37,741][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
  DefaultMemoryPolicySize=12753MB
[2017-08-25 19:44:37,741][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
  SystemCacheInitialSize=40MB
[2017-08-25 19:44:37,741][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
  SystemCacheMaxSize=100MB
[2017-08-25 19:44:37,741][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
[2017-08-25 19:44:37,741][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]
Other ignite instanses on the server require: 12853MB
[2017-08-25 19:44:37,741][WARN
][disco-event-worker-#29%internal.GridHomePathSelfTest0%][GridDiscoveryManager]


2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:

> I suppose we should not forget JVM heap size and suggest to reduce both
> options
>
> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <[hidden email]>
> wrote:
>
> > Igor, I would change the message. How about this:
> >
> > Required RAM size is larger than total physical memory available for OS.
> > > Please change MemoryConfiguration.WhichProperty and
> > > MemoryPolicyConfiguration.WhichProperty to decrease memory allocated
> for
> > > each node.
> >
> >
> > Also, can we calculate what the memory size allocated for each node
> should
> > be? In that case we should suggest it.
> >
> > D.
> >
> >
> > On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <[hidden email]>
> > wrote:
> >
> > > What do you, guys think about next warning?
> > >
> > > [2017-08-25 17:17:04,718][INFO
> > > ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePathSelfTest0]
> > > System cache's MemoryPolicy size is configured to 40 MB. Use
> > > MemoryConfiguration.systemCacheMemorySize property to change the
> > setting.
> > > [2017-08-25 17:17:04,718][WARN
> > > ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePathSelfTest0]
> > >
> > > >>> Required RAM size is larger than total physical memory available
> for
> > > OS.
> > > >>> Check your configuration to avoid swap partition usage.
> > > >>> Use MemoryConfiguration and MemoryPolicyConfiguration to change the
> > > settings.
> > > >>> Physical memory [required=16397MB, available=15942MB]
> > >
> > > [2017-08-25 17:17:04,726][WARN
> > > ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePathSelfTest0]
> > > Peer class loading is enabled (disable it in production for performance
> > and
> > > deployment consistency reasons)
> > > [2017-08-25 17:17:04,726][INFO
> > > ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePathSelfTest0]
> > > Configured caches [in 'sysMemPlc' memoryPolicy: ['ignite-sys-cache']]
> > > [2017-08-25 17:17:04,731][INFO
> > > ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePathSelfTest0]
> > > 3-rd party licenses can be found at:
> > > /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
> > >
> > >
> > > 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> > >
> > > > Agree, let's release new version including tickets mentioned by Denis
> > and
> > > > Ivan.
> > > >
> > > > --Yakov
> > > >
> > >
> >
>
>
>
> --
> Sergey Kozlov
> GridGain Systems
> www.gridgain.com
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

gvvinblade
This message appears on topology change in case the available memory is
exceeded

2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]>:

> An example of current impl:
>
>
> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]
> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory for
> current process [required=29251MB, available=15942MB].
> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.defaultMemoryPolicySize
> to decrease memory allocated for each node.
> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]
> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]   DefaultMemoryPolicySize=
> 12753MB
> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]
> SystemCacheInitialSize=40MB
> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]   SystemCacheMaxSize=100MB
> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]
> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite instanses on
> the server require: 12853MB
> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> GridHomePathSelfTest0%][GridDiscoveryManager]
>
>
> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
>
>> I suppose we should not forget JVM heap size and suggest to reduce both
>> options
>>
>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <[hidden email]
>> >
>> wrote:
>>
>> > Igor, I would change the message. How about this:
>> >
>> > Required RAM size is larger than total physical memory available for OS.
>> > > Please change MemoryConfiguration.WhichProperty and
>> > > MemoryPolicyConfiguration.WhichProperty to decrease memory allocated
>> for
>> > > each node.
>> >
>> >
>> > Also, can we calculate what the memory size allocated for each node
>> should
>> > be? In that case we should suggest it.
>> >
>> > D.
>> >
>> >
>> > On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <[hidden email]
>> >
>> > wrote:
>> >
>> > > What do you, guys think about next warning?
>> > >
>> > > [2017-08-25 17:17:04,718][INFO
>> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>> hSelfTest0]
>> > > System cache's MemoryPolicy size is configured to 40 MB. Use
>> > > MemoryConfiguration.systemCacheMemorySize property to change the
>> > setting.
>> > > [2017-08-25 17:17:04,718][WARN
>> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>> hSelfTest0]
>> > >
>> > > >>> Required RAM size is larger than total physical memory available
>> for
>> > > OS.
>> > > >>> Check your configuration to avoid swap partition usage.
>> > > >>> Use MemoryConfiguration and MemoryPolicyConfiguration to change
>> the
>> > > settings.
>> > > >>> Physical memory [required=16397MB, available=15942MB]
>> > >
>> > > [2017-08-25 17:17:04,726][WARN
>> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>> hSelfTest0]
>> > > Peer class loading is enabled (disable it in production for
>> performance
>> > and
>> > > deployment consistency reasons)
>> > > [2017-08-25 17:17:04,726][INFO
>> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>> hSelfTest0]
>> > > Configured caches [in 'sysMemPlc' memoryPolicy: ['ignite-sys-cache']]
>> > > [2017-08-25 17:17:04,731][INFO
>> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>> hSelfTest0]
>> > > 3-rd party licenses can be found at:
>> > > /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
>> > >
>> > >
>> > > 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>> > >
>> > > > Agree, let's release new version including tickets mentioned by
>> Denis
>> > and
>> > > > Ivan.
>> > > >
>> > > > --Yakov
>> > > >
>> > >
>> >
>>
>>
>>
>> --
>> Sergey Kozlov
>> GridGain Systems
>> www.gridgain.com
>>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dsetrakyan
Igor, what is this flood of WARN messaging coming after the text? Are we
really going to print this whole thing out?

On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <[hidden email]>
wrote:

> This message appears on topology change in case the available memory is
> exceeded
>
> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]>:
>
> > An example of current impl:
> >
> >
> > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]
> > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory for
> > current process [required=29251MB, available=15942MB].
> > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> > MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
> defaultMemoryPolicySize
> > to decrease memory allocated for each node.
> > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]
> > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
> > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
> > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
> > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]   DefaultMemoryPolicySize=
> > 12753MB
> > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]
> > SystemCacheInitialSize=40MB
> > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]   SystemCacheMaxSize=100MB
> > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]
> > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite instanses on
> > the server require: 12853MB
> > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > GridHomePathSelfTest0%][GridDiscoveryManager]
> >
> >
> > 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
> >
> >> I suppose we should not forget JVM heap size and suggest to reduce both
> >> options
> >>
> >> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> [hidden email]
> >> >
> >> wrote:
> >>
> >> > Igor, I would change the message. How about this:
> >> >
> >> > Required RAM size is larger than total physical memory available for
> OS.
> >> > > Please change MemoryConfiguration.WhichProperty and
> >> > > MemoryPolicyConfiguration.WhichProperty to decrease memory
> allocated
> >> for
> >> > > each node.
> >> >
> >> >
> >> > Also, can we calculate what the memory size allocated for each node
> >> should
> >> > be? In that case we should suggest it.
> >> >
> >> > D.
> >> >
> >> >
> >> > On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> [hidden email]
> >> >
> >> > wrote:
> >> >
> >> > > What do you, guys think about next warning?
> >> > >
> >> > > [2017-08-25 17:17:04,718][INFO
> >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >> hSelfTest0]
> >> > > System cache's MemoryPolicy size is configured to 40 MB. Use
> >> > > MemoryConfiguration.systemCacheMemorySize property to change the
> >> > setting.
> >> > > [2017-08-25 17:17:04,718][WARN
> >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >> hSelfTest0]
> >> > >
> >> > > >>> Required RAM size is larger than total physical memory available
> >> for
> >> > > OS.
> >> > > >>> Check your configuration to avoid swap partition usage.
> >> > > >>> Use MemoryConfiguration and MemoryPolicyConfiguration to change
> >> the
> >> > > settings.
> >> > > >>> Physical memory [required=16397MB, available=15942MB]
> >> > >
> >> > > [2017-08-25 17:17:04,726][WARN
> >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >> hSelfTest0]
> >> > > Peer class loading is enabled (disable it in production for
> >> performance
> >> > and
> >> > > deployment consistency reasons)
> >> > > [2017-08-25 17:17:04,726][INFO
> >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >> hSelfTest0]
> >> > > Configured caches [in 'sysMemPlc' memoryPolicy:
> ['ignite-sys-cache']]
> >> > > [2017-08-25 17:17:04,731][INFO
> >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >> hSelfTest0]
> >> > > 3-rd party licenses can be found at:
> >> > > /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
> >> > >
> >> > >
> >> > > 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> >> > >
> >> > > > Agree, let's release new version including tickets mentioned by
> >> Denis
> >> > and
> >> > > > Ivan.
> >> > > >
> >> > > > --Yakov
> >> > > >
> >> > >
> >> >
> >>
> >>
> >>
> >> --
> >> Sergey Kozlov
> >> GridGain Systems
> >> www.gridgain.com
> >>
> >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

gvvinblade
The message without logging layout:

Not enough memory for current process [required=29251MB, available=15942MB].
Please change MemoryConfiguration.systemCacheMaxSize and
MemoryConfiguration.defaultMemoryPolicySize to decrease memory allocated
for each node.

Current settings:
   HeapInit=250MB
   HeapMax=3543MB
   DefaultMemoryPolicySize=12753MB
   SystemCacheInitialSize=40MB
   SystemCacheMaxSize=100MB

Other ignite instanses on the server require: 12853MB

I think it's make sense to describe what these numbers are consist of.

We simple say which parameters have an impact on how much memory the
instance needs and their (parameters) actual values.

Also we notice that more than one Ignite instance are ran on the server or
workstation and it also consumes memory.

25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
[hidden email]> написал:

> Igor, what is this flood of WARN messaging coming after the text? Are we
> really going to print this whole thing out?
>
> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <[hidden email]>
> wrote:
>
> > This message appears on topology change in case the available memory is
> > exceeded
> >
> > 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]>:
> >
> > > An example of current impl:
> > >
> > >
> > > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]
> > > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory for
> > > current process [required=29251MB, available=15942MB].
> > > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> > > MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
> > defaultMemoryPolicySize
> > > to decrease memory allocated for each node.
> > > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]
> > > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
> > > [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
> > > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
> > > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]
>  DefaultMemoryPolicySize=
> > > 12753MB
> > > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]
> > > SystemCacheInitialSize=40MB
> > > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]
>  SystemCacheMaxSize=100MB
> > > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]
> > > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite instanses
> on
> > > the server require: 12853MB
> > > [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > > GridHomePathSelfTest0%][GridDiscoveryManager]
> > >
> > >
> > > 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
> > >
> > >> I suppose we should not forget JVM heap size and suggest to reduce
> both
> > >> options
> > >>
> > >> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> > [hidden email]
> > >> >
> > >> wrote:
> > >>
> > >> > Igor, I would change the message. How about this:
> > >> >
> > >> > Required RAM size is larger than total physical memory available for
> > OS.
> > >> > > Please change MemoryConfiguration.WhichProperty and
> > >> > > MemoryPolicyConfiguration.WhichProperty to decrease memory
> > allocated
> > >> for
> > >> > > each node.
> > >> >
> > >> >
> > >> > Also, can we calculate what the memory size allocated for each node
> > >> should
> > >> > be? In that case we should suggest it.
> > >> >
> > >> > D.
> > >> >
> > >> >
> > >> > On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> > [hidden email]
> > >> >
> > >> > wrote:
> > >> >
> > >> > > What do you, guys think about next warning?
> > >> > >
> > >> > > [2017-08-25 17:17:04,718][INFO
> > >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >> hSelfTest0]
> > >> > > System cache's MemoryPolicy size is configured to 40 MB. Use
> > >> > > MemoryConfiguration.systemCacheMemorySize property to change the
> > >> > setting.
> > >> > > [2017-08-25 17:17:04,718][WARN
> > >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >> hSelfTest0]
> > >> > >
> > >> > > >>> Required RAM size is larger than total physical memory
> available
> > >> for
> > >> > > OS.
> > >> > > >>> Check your configuration to avoid swap partition usage.
> > >> > > >>> Use MemoryConfiguration and MemoryPolicyConfiguration to
> change
> > >> the
> > >> > > settings.
> > >> > > >>> Physical memory [required=16397MB, available=15942MB]
> > >> > >
> > >> > > [2017-08-25 17:17:04,726][WARN
> > >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >> hSelfTest0]
> > >> > > Peer class loading is enabled (disable it in production for
> > >> performance
> > >> > and
> > >> > > deployment consistency reasons)
> > >> > > [2017-08-25 17:17:04,726][INFO
> > >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >> hSelfTest0]
> > >> > > Configured caches [in 'sysMemPlc' memoryPolicy:
> > ['ignite-sys-cache']]
> > >> > > [2017-08-25 17:17:04,731][INFO
> > >> > > ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >> hSelfTest0]
> > >> > > 3-rd party licenses can be found at:
> > >> > > /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
> > >> > >
> > >> > >
> > >> > > 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> > >> > >
> > >> > > > Agree, let's release new version including tickets mentioned by
> > >> Denis
> > >> > and
> > >> > > > Ivan.
> > >> > > >
> > >> > > > --Yakov
> > >> > > >
> > >> > >
> > >> >
> > >>
> > >>
> > >>
> > >> --
> > >> Sergey Kozlov
> > >> GridGain Systems
> > >> www.gridgain.com
> > >>
> > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dmagda
Igor,

Let me suggest this format.

---------------------------------------------
Excessive memory usage by Ignite node process (performance may drop) [requested=29251MB, available=15942MB]

Please tune the following settings:
  [MemoryConfiguration.defaultMemoryPolicySize = suggested value]
  MemoryConfiguration.systemCacheMaxSize = suggested value
  [MemoryPolicyConfiguration.maxSize for {policy_name_1} = suggested value]
  [MemoryPolicyConfiguration.maxSize for {policy_name_2} = suggested value]

Current settings:
   [DefaultMemoryPolicySize = value]
   [{policy_name_1} size = value]
   [{policy_name_1} size = value]
   SystemCacheInitialSize = value
   SystemCacheMaxSize = value
   Java Heap Init Size = value
   Java Heap Max Size = value

The overall memory usage by all Ignite nodes on the host: value
-------------------------------------------

Records in […] are optional. If custom memory policy is not set or the default memory policy is overridden the output will miss some of the rows.

As for systemCacheMaxSize, it should be show ONLY if the parameter was set explicitly by user code. Otherwise, the platform should be wise enough to instantiate it properly depending on the host memory usage.


Denis
   

> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <[hidden email]> wrote:
>
> The message without logging layout:
>
> Not enough memory for current process [required=29251MB, available=15942MB].
> Please change MemoryConfiguration.systemCacheMaxSize and
> MemoryConfiguration.defaultMemoryPolicySize to decrease memory allocated
> for each node.
>
> Current settings:
>   HeapInit=250MB
>   HeapMax=3543MB
>   DefaultMemoryPolicySize=12753MB
>   SystemCacheInitialSize=40MB
>   SystemCacheMaxSize=100MB
>
> Other ignite instanses on the server require: 12853MB
>
> I think it's make sense to describe what these numbers are consist of.
>
> We simple say which parameters have an impact on how much memory the
> instance needs and their (parameters) actual values.
>
> Also we notice that more than one Ignite instance are ran on the server or
> workstation and it also consumes memory.
>
> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> [hidden email]> написал:
>
>> Igor, what is this flood of WARN messaging coming after the text? Are we
>> really going to print this whole thing out?
>>
>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <[hidden email]>
>> wrote:
>>
>>> This message appears on topology change in case the available memory is
>>> exceeded
>>>
>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]>:
>>>
>>>> An example of current impl:
>>>>
>>>>
>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory for
>>>> current process [required=29251MB, available=15942MB].
>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
>>>> MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
>>> defaultMemoryPolicySize
>>>> to decrease memory allocated for each node.
>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>> DefaultMemoryPolicySize=
>>>> 12753MB
>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>> SystemCacheInitialSize=40MB
>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>> SystemCacheMaxSize=100MB
>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite instanses
>> on
>>>> the server require: 12853MB
>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>>
>>>>
>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
>>>>
>>>>> I suppose we should not forget JVM heap size and suggest to reduce
>> both
>>>>> options
>>>>>
>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
>>> [hidden email]
>>>>>>
>>>>> wrote:
>>>>>
>>>>>> Igor, I would change the message. How about this:
>>>>>>
>>>>>> Required RAM size is larger than total physical memory available for
>>> OS.
>>>>>>> Please change MemoryConfiguration.WhichProperty and
>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease memory
>>> allocated
>>>>> for
>>>>>>> each node.
>>>>>>
>>>>>>
>>>>>> Also, can we calculate what the memory size allocated for each node
>>>>> should
>>>>>> be? In that case we should suggest it.
>>>>>>
>>>>>> D.
>>>>>>
>>>>>>
>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
>>> [hidden email]
>>>>>>
>>>>>> wrote:
>>>>>>
>>>>>>> What do you, guys think about next warning?
>>>>>>>
>>>>>>> [2017-08-25 17:17:04,718][INFO
>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>> hSelfTest0]
>>>>>>> System cache's MemoryPolicy size is configured to 40 MB. Use
>>>>>>> MemoryConfiguration.systemCacheMemorySize property to change the
>>>>>> setting.
>>>>>>> [2017-08-25 17:17:04,718][WARN
>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>> hSelfTest0]
>>>>>>>
>>>>>>>>>> Required RAM size is larger than total physical memory
>> available
>>>>> for
>>>>>>> OS.
>>>>>>>>>> Check your configuration to avoid swap partition usage.
>>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration to
>> change
>>>>> the
>>>>>>> settings.
>>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
>>>>>>>
>>>>>>> [2017-08-25 17:17:04,726][WARN
>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>> hSelfTest0]
>>>>>>> Peer class loading is enabled (disable it in production for
>>>>> performance
>>>>>> and
>>>>>>> deployment consistency reasons)
>>>>>>> [2017-08-25 17:17:04,726][INFO
>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>> hSelfTest0]
>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
>>> ['ignite-sys-cache']]
>>>>>>> [2017-08-25 17:17:04,731][INFO
>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>> hSelfTest0]
>>>>>>> 3-rd party licenses can be found at:
>>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
>>>>>>>
>>>>>>>
>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>>>>>>>
>>>>>>>> Agree, let's release new version including tickets mentioned by
>>>>> Denis
>>>>>> and
>>>>>>>> Ivan.
>>>>>>>>
>>>>>>>> --Yakov
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Sergey Kozlov
>>>>> GridGain Systems
>>>>> www.gridgain.com
>>>>>
>>>>
>>>>
>>>
>>

Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dsetrakyan
I like the format proposed by Denis, very clear.

However, I also do not understand why a user should change the size of some
system cache. How would a user ever know what value to put there? This
value should be configured by Ignite automatically.

D.

On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <[hidden email]> wrote:

> Igor,
>
> Let me suggest this format.
>
> ---------------------------------------------
> Excessive memory usage by Ignite node process (performance may drop)
> [requested=29251MB, available=15942MB]
>
> Please tune the following settings:
>   [MemoryConfiguration.defaultMemoryPolicySize = suggested value]
>   MemoryConfiguration.systemCacheMaxSize = suggested value
>   [MemoryPolicyConfiguration.maxSize for {policy_name_1} = suggested
> value]
>   [MemoryPolicyConfiguration.maxSize for {policy_name_2} = suggested
> value]
>
> Current settings:
>    [DefaultMemoryPolicySize = value]
>    [{policy_name_1} size = value]
>    [{policy_name_1} size = value]
>    SystemCacheInitialSize = value
>    SystemCacheMaxSize = value
>    Java Heap Init Size = value
>    Java Heap Max Size = value
>
> The overall memory usage by all Ignite nodes on the host: value
> -------------------------------------------
>
> Records in […] are optional. If custom memory policy is not set or the
> default memory policy is overridden the output will miss some of the rows.
>
> As for systemCacheMaxSize, it should be show ONLY if the parameter was set
> explicitly by user code. Otherwise, the platform should be wise enough to
> instantiate it properly depending on the host memory usage.
>
> —
> Denis
>
> > On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <[hidden email]>
> wrote:
> >
> > The message without logging layout:
> >
> > Not enough memory for current process [required=29251MB,
> available=15942MB].
> > Please change MemoryConfiguration.systemCacheMaxSize and
> > MemoryConfiguration.defaultMemoryPolicySize to decrease memory allocated
> > for each node.
> >
> > Current settings:
> >   HeapInit=250MB
> >   HeapMax=3543MB
> >   DefaultMemoryPolicySize=12753MB
> >   SystemCacheInitialSize=40MB
> >   SystemCacheMaxSize=100MB
> >
> > Other ignite instanses on the server require: 12853MB
> >
> > I think it's make sense to describe what these numbers are consist of.
> >
> > We simple say which parameters have an impact on how much memory the
> > instance needs and their (parameters) actual values.
> >
> > Also we notice that more than one Ignite instance are ran on the server
> or
> > workstation and it also consumes memory.
> >
> > 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> > [hidden email]> написал:
> >
> >> Igor, what is this flood of WARN messaging coming after the text? Are we
> >> really going to print this whole thing out?
> >>
> >> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <[hidden email]
> >
> >> wrote:
> >>
> >>> This message appears on topology change in case the available memory is
> >>> exceeded
> >>>
> >>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]>:
> >>>
> >>>> An example of current impl:
> >>>>
> >>>>
> >>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory for
> >>>> current process [required=29251MB, available=15942MB].
> >>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> >>>> MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
> >>> defaultMemoryPolicySize
> >>>> to decrease memory allocated for each node.
> >>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
> >>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
> >>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
> >>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >> DefaultMemoryPolicySize=
> >>>> 12753MB
> >>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>> SystemCacheInitialSize=40MB
> >>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >> SystemCacheMaxSize=100MB
> >>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite instanses
> >> on
> >>>> the server require: 12853MB
> >>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>>
> >>>>
> >>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
> >>>>
> >>>>> I suppose we should not forget JVM heap size and suggest to reduce
> >> both
> >>>>> options
> >>>>>
> >>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> >>> [hidden email]
> >>>>>>
> >>>>> wrote:
> >>>>>
> >>>>>> Igor, I would change the message. How about this:
> >>>>>>
> >>>>>> Required RAM size is larger than total physical memory available for
> >>> OS.
> >>>>>>> Please change MemoryConfiguration.WhichProperty and
> >>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease memory
> >>> allocated
> >>>>> for
> >>>>>>> each node.
> >>>>>>
> >>>>>>
> >>>>>> Also, can we calculate what the memory size allocated for each node
> >>>>> should
> >>>>>> be? In that case we should suggest it.
> >>>>>>
> >>>>>> D.
> >>>>>>
> >>>>>>
> >>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> >>> [hidden email]
> >>>>>>
> >>>>>> wrote:
> >>>>>>
> >>>>>>> What do you, guys think about next warning?
> >>>>>>>
> >>>>>>> [2017-08-25 17:17:04,718][INFO
> >>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>> hSelfTest0]
> >>>>>>> System cache's MemoryPolicy size is configured to 40 MB. Use
> >>>>>>> MemoryConfiguration.systemCacheMemorySize property to change the
> >>>>>> setting.
> >>>>>>> [2017-08-25 17:17:04,718][WARN
> >>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>> hSelfTest0]
> >>>>>>>
> >>>>>>>>>> Required RAM size is larger than total physical memory
> >> available
> >>>>> for
> >>>>>>> OS.
> >>>>>>>>>> Check your configuration to avoid swap partition usage.
> >>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration to
> >> change
> >>>>> the
> >>>>>>> settings.
> >>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
> >>>>>>>
> >>>>>>> [2017-08-25 17:17:04,726][WARN
> >>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>> hSelfTest0]
> >>>>>>> Peer class loading is enabled (disable it in production for
> >>>>> performance
> >>>>>> and
> >>>>>>> deployment consistency reasons)
> >>>>>>> [2017-08-25 17:17:04,726][INFO
> >>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>> hSelfTest0]
> >>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
> >>> ['ignite-sys-cache']]
> >>>>>>> [2017-08-25 17:17:04,731][INFO
> >>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>> hSelfTest0]
> >>>>>>> 3-rd party licenses can be found at:
> >>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
> >>>>>>>
> >>>>>>>
> >>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> >>>>>>>
> >>>>>>>> Agree, let's release new version including tickets mentioned by
> >>>>> Denis
> >>>>>> and
> >>>>>>>> Ivan.
> >>>>>>>>
> >>>>>>>> --Yakov
> >>>>>>>>
> >>>>>>>
> >>>>>>
> >>>>>
> >>>>>
> >>>>>
> >>>>> --
> >>>>> Sergey Kozlov
> >>>>> GridGain Systems
> >>>>> www.gridgain.com
> >>>>>
> >>>>
> >>>>
> >>>
> >>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dmagda
Guys,

ML lib profile is missing in 2.1 release! That must be fixed and rolled out in this emergency release:
https://issues.apache.org/jira/browse/IGNITE-6193 <https://issues.apache.org/jira/browse/IGNITE-6193>

Oleg, Yuri, please step in and handle the issue.

BTW, who is considered to be the release manager of this release?


Denis

> On Aug 25, 2017, at 2:29 PM, Dmitriy Setrakyan <[hidden email]> wrote:
>
> I like the format proposed by Denis, very clear.
>
> However, I also do not understand why a user should change the size of some
> system cache. How would a user ever know what value to put there? This
> value should be configured by Ignite automatically.
>
> D.
>
> On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <[hidden email]> wrote:
>
>> Igor,
>>
>> Let me suggest this format.
>>
>> ---------------------------------------------
>> Excessive memory usage by Ignite node process (performance may drop)
>> [requested=29251MB, available=15942MB]
>>
>> Please tune the following settings:
>>  [MemoryConfiguration.defaultMemoryPolicySize = suggested value]
>>  MemoryConfiguration.systemCacheMaxSize = suggested value
>>  [MemoryPolicyConfiguration.maxSize for {policy_name_1} = suggested
>> value]
>>  [MemoryPolicyConfiguration.maxSize for {policy_name_2} = suggested
>> value]
>>
>> Current settings:
>>   [DefaultMemoryPolicySize = value]
>>   [{policy_name_1} size = value]
>>   [{policy_name_1} size = value]
>>   SystemCacheInitialSize = value
>>   SystemCacheMaxSize = value
>>   Java Heap Init Size = value
>>   Java Heap Max Size = value
>>
>> The overall memory usage by all Ignite nodes on the host: value
>> -------------------------------------------
>>
>> Records in […] are optional. If custom memory policy is not set or the
>> default memory policy is overridden the output will miss some of the rows.
>>
>> As for systemCacheMaxSize, it should be show ONLY if the parameter was set
>> explicitly by user code. Otherwise, the platform should be wise enough to
>> instantiate it properly depending on the host memory usage.
>>
>> —
>> Denis
>>
>>> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <[hidden email]>
>> wrote:
>>>
>>> The message without logging layout:
>>>
>>> Not enough memory for current process [required=29251MB,
>> available=15942MB].
>>> Please change MemoryConfiguration.systemCacheMaxSize and
>>> MemoryConfiguration.defaultMemoryPolicySize to decrease memory allocated
>>> for each node.
>>>
>>> Current settings:
>>>  HeapInit=250MB
>>>  HeapMax=3543MB
>>>  DefaultMemoryPolicySize=12753MB
>>>  SystemCacheInitialSize=40MB
>>>  SystemCacheMaxSize=100MB
>>>
>>> Other ignite instanses on the server require: 12853MB
>>>
>>> I think it's make sense to describe what these numbers are consist of.
>>>
>>> We simple say which parameters have an impact on how much memory the
>>> instance needs and their (parameters) actual values.
>>>
>>> Also we notice that more than one Ignite instance are ran on the server
>> or
>>> workstation and it also consumes memory.
>>>
>>> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
>>> [hidden email]> написал:
>>>
>>>> Igor, what is this flood of WARN messaging coming after the text? Are we
>>>> really going to print this whole thing out?
>>>>
>>>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <[hidden email]
>>>
>>>> wrote:
>>>>
>>>>> This message appears on topology change in case the available memory is
>>>>> exceeded
>>>>>
>>>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]>:
>>>>>
>>>>>> An example of current impl:
>>>>>>
>>>>>>
>>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory for
>>>>>> current process [required=29251MB, available=15942MB].
>>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
>>>>>> MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
>>>>> defaultMemoryPolicySize
>>>>>> to decrease memory allocated for each node.
>>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
>>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
>>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
>>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>> DefaultMemoryPolicySize=
>>>>>> 12753MB
>>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>>>> SystemCacheInitialSize=40MB
>>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>> SystemCacheMaxSize=100MB
>>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite instanses
>>>> on
>>>>>> the server require: 12853MB
>>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
>>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>>>>>>
>>>>>>
>>>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
>>>>>>
>>>>>>> I suppose we should not forget JVM heap size and suggest to reduce
>>>> both
>>>>>>> options
>>>>>>>
>>>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
>>>>> [hidden email]
>>>>>>>>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> Igor, I would change the message. How about this:
>>>>>>>>
>>>>>>>> Required RAM size is larger than total physical memory available for
>>>>> OS.
>>>>>>>>> Please change MemoryConfiguration.WhichProperty and
>>>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease memory
>>>>> allocated
>>>>>>> for
>>>>>>>>> each node.
>>>>>>>>
>>>>>>>>
>>>>>>>> Also, can we calculate what the memory size allocated for each node
>>>>>>> should
>>>>>>>> be? In that case we should suggest it.
>>>>>>>>
>>>>>>>> D.
>>>>>>>>
>>>>>>>>
>>>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
>>>>> [hidden email]
>>>>>>>>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>>> What do you, guys think about next warning?
>>>>>>>>>
>>>>>>>>> [2017-08-25 17:17:04,718][INFO
>>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>>>> hSelfTest0]
>>>>>>>>> System cache's MemoryPolicy size is configured to 40 MB. Use
>>>>>>>>> MemoryConfiguration.systemCacheMemorySize property to change the
>>>>>>>> setting.
>>>>>>>>> [2017-08-25 17:17:04,718][WARN
>>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>>>> hSelfTest0]
>>>>>>>>>
>>>>>>>>>>>> Required RAM size is larger than total physical memory
>>>> available
>>>>>>> for
>>>>>>>>> OS.
>>>>>>>>>>>> Check your configuration to avoid swap partition usage.
>>>>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration to
>>>> change
>>>>>>> the
>>>>>>>>> settings.
>>>>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
>>>>>>>>>
>>>>>>>>> [2017-08-25 17:17:04,726][WARN
>>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>>>> hSelfTest0]
>>>>>>>>> Peer class loading is enabled (disable it in production for
>>>>>>> performance
>>>>>>>> and
>>>>>>>>> deployment consistency reasons)
>>>>>>>>> [2017-08-25 17:17:04,726][INFO
>>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>>>> hSelfTest0]
>>>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
>>>>> ['ignite-sys-cache']]
>>>>>>>>> [2017-08-25 17:17:04,731][INFO
>>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
>>>>>>> hSelfTest0]
>>>>>>>>> 3-rd party licenses can be found at:
>>>>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>>>>>>>>>
>>>>>>>>>> Agree, let's release new version including tickets mentioned by
>>>>>>> Denis
>>>>>>>> and
>>>>>>>>>> Ivan.
>>>>>>>>>>
>>>>>>>>>> --Yakov
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Sergey Kozlov
>>>>>>> GridGain Systems
>>>>>>> www.gridgain.com
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>
>>

Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

gvvinblade
One more example of possible warning:

-----------------------------------------------------
Excessive memory usage by Ignite node process (performance may drop)
[requested=44613MB, available=15942MB].

Please tune the folowing settings as suggested:
  MemoryPolicyConfiguration.initialSize for bigPlc: 8102MB
  MemoryPolicyConfiguration.maxSize     for bigPlc: 8102MB
  MemoryPolicyConfiguration.initialSize for dfltPlc: 100MB
  MemoryPolicyConfiguration.maxSize     for dfltPlc: 100MB

Current settings:
  Java Heap  maxSize: 3543MB
  Java Heap initSize: 250MB
  MemoryPolicyConfiguration.initialSize for bigPlc: 256MB
  MemoryPolicyConfiguration.maxSize     for bigPlc: 40960MB
  MemoryPolicyConfiguration.initialSize for dfltPlc: 10MB
  MemoryPolicyConfiguration.maxSize     for dfltPlc: 10MB
  The overall expected memory usage by all Ignite nodes on the host: 44613MB
-----------------------------------------------------

Your thoughts?

2017-08-28 5:06 GMT+03:00 Denis Magda <[hidden email]>:

> Guys,
>
> ML lib profile is missing in 2.1 release! That must be fixed and rolled
> out in this emergency release:
> https://issues.apache.org/jira/browse/IGNITE-6193 <
> https://issues.apache.org/jira/browse/IGNITE-6193>
>
> Oleg, Yuri, please step in and handle the issue.
>
> BTW, who is considered to be the release manager of this release?
>
> —
> Denis
>
> > On Aug 25, 2017, at 2:29 PM, Dmitriy Setrakyan <[hidden email]>
> wrote:
> >
> > I like the format proposed by Denis, very clear.
> >
> > However, I also do not understand why a user should change the size of
> some
> > system cache. How would a user ever know what value to put there? This
> > value should be configured by Ignite automatically.
> >
> > D.
> >
> > On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <[hidden email]> wrote:
> >
> >> Igor,
> >>
> >> Let me suggest this format.
> >>
> >> ---------------------------------------------
> >> Excessive memory usage by Ignite node process (performance may drop)
> >> [requested=29251MB, available=15942MB]
> >>
> >> Please tune the following settings:
> >>  [MemoryConfiguration.defaultMemoryPolicySize = suggested value]
> >>  MemoryConfiguration.systemCacheMaxSize = suggested value
> >>  [MemoryPolicyConfiguration.maxSize for {policy_name_1} = suggested
> >> value]
> >>  [MemoryPolicyConfiguration.maxSize for {policy_name_2} = suggested
> >> value]
> >>
> >> Current settings:
> >>   [DefaultMemoryPolicySize = value]
> >>   [{policy_name_1} size = value]
> >>   [{policy_name_1} size = value]
> >>   SystemCacheInitialSize = value
> >>   SystemCacheMaxSize = value
> >>   Java Heap Init Size = value
> >>   Java Heap Max Size = value
> >>
> >> The overall memory usage by all Ignite nodes on the host: value
> >> -------------------------------------------
> >>
> >> Records in […] are optional. If custom memory policy is not set or the
> >> default memory policy is overridden the output will miss some of the
> rows.
> >>
> >> As for systemCacheMaxSize, it should be show ONLY if the parameter was
> set
> >> explicitly by user code. Otherwise, the platform should be wise enough
> to
> >> instantiate it properly depending on the host memory usage.
> >>
> >> —
> >> Denis
> >>
> >>> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <[hidden email]>
> >> wrote:
> >>>
> >>> The message without logging layout:
> >>>
> >>> Not enough memory for current process [required=29251MB,
> >> available=15942MB].
> >>> Please change MemoryConfiguration.systemCacheMaxSize and
> >>> MemoryConfiguration.defaultMemoryPolicySize to decrease memory
> allocated
> >>> for each node.
> >>>
> >>> Current settings:
> >>>  HeapInit=250MB
> >>>  HeapMax=3543MB
> >>>  DefaultMemoryPolicySize=12753MB
> >>>  SystemCacheInitialSize=40MB
> >>>  SystemCacheMaxSize=100MB
> >>>
> >>> Other ignite instanses on the server require: 12853MB
> >>>
> >>> I think it's make sense to describe what these numbers are consist of.
> >>>
> >>> We simple say which parameters have an impact on how much memory the
> >>> instance needs and their (parameters) actual values.
> >>>
> >>> Also we notice that more than one Ignite instance are ran on the server
> >> or
> >>> workstation and it also consumes memory.
> >>>
> >>> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> >>> [hidden email]> написал:
> >>>
> >>>> Igor, what is this flood of WARN messaging coming after the text? Are
> we
> >>>> really going to print this whole thing out?
> >>>>
> >>>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <
> [hidden email]
> >>>
> >>>> wrote:
> >>>>
> >>>>> This message appears on topology change in case the available memory
> is
> >>>>> exceeded
> >>>>>
> >>>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]>:
> >>>>>
> >>>>>> An example of current impl:
> >>>>>>
> >>>>>>
> >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory for
> >>>>>> current process [required=29251MB, available=15942MB].
> >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> >>>>>> MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
> >>>>> defaultMemoryPolicySize
> >>>>>> to decrease memory allocated for each node.
> >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
> >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
> >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
> >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>> DefaultMemoryPolicySize=
> >>>>>> 12753MB
> >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>>>> SystemCacheInitialSize=40MB
> >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>> SystemCacheMaxSize=100MB
> >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite
> instanses
> >>>> on
> >>>>>> the server require: 12853MB
> >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >>>>>>
> >>>>>>
> >>>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
> >>>>>>
> >>>>>>> I suppose we should not forget JVM heap size and suggest to reduce
> >>>> both
> >>>>>>> options
> >>>>>>>
> >>>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> >>>>> [hidden email]
> >>>>>>>>
> >>>>>>> wrote:
> >>>>>>>
> >>>>>>>> Igor, I would change the message. How about this:
> >>>>>>>>
> >>>>>>>> Required RAM size is larger than total physical memory available
> for
> >>>>> OS.
> >>>>>>>>> Please change MemoryConfiguration.WhichProperty and
> >>>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease memory
> >>>>> allocated
> >>>>>>> for
> >>>>>>>>> each node.
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> Also, can we calculate what the memory size allocated for each
> node
> >>>>>>> should
> >>>>>>>> be? In that case we should suggest it.
> >>>>>>>>
> >>>>>>>> D.
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> >>>>> [hidden email]
> >>>>>>>>
> >>>>>>>> wrote:
> >>>>>>>>
> >>>>>>>>> What do you, guys think about next warning?
> >>>>>>>>>
> >>>>>>>>> [2017-08-25 17:17:04,718][INFO
> >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>>>> hSelfTest0]
> >>>>>>>>> System cache's MemoryPolicy size is configured to 40 MB. Use
> >>>>>>>>> MemoryConfiguration.systemCacheMemorySize property to change the
> >>>>>>>> setting.
> >>>>>>>>> [2017-08-25 17:17:04,718][WARN
> >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>>>> hSelfTest0]
> >>>>>>>>>
> >>>>>>>>>>>> Required RAM size is larger than total physical memory
> >>>> available
> >>>>>>> for
> >>>>>>>>> OS.
> >>>>>>>>>>>> Check your configuration to avoid swap partition usage.
> >>>>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration to
> >>>> change
> >>>>>>> the
> >>>>>>>>> settings.
> >>>>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
> >>>>>>>>>
> >>>>>>>>> [2017-08-25 17:17:04,726][WARN
> >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>>>> hSelfTest0]
> >>>>>>>>> Peer class loading is enabled (disable it in production for
> >>>>>>> performance
> >>>>>>>> and
> >>>>>>>>> deployment consistency reasons)
> >>>>>>>>> [2017-08-25 17:17:04,726][INFO
> >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>>>> hSelfTest0]
> >>>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
> >>>>> ['ignite-sys-cache']]
> >>>>>>>>> [2017-08-25 17:17:04,731][INFO
> >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> >>>>>>> hSelfTest0]
> >>>>>>>>> 3-rd party licenses can be found at:
> >>>>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/licenses
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> >>>>>>>>>
> >>>>>>>>>> Agree, let's release new version including tickets mentioned by
> >>>>>>> Denis
> >>>>>>>> and
> >>>>>>>>>> Ivan.
> >>>>>>>>>>
> >>>>>>>>>> --Yakov
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> --
> >>>>>>> Sergey Kozlov
> >>>>>>> GridGain Systems
> >>>>>>> www.gridgain.com
> >>>>>>>
> >>>>>>
> >>>>>>
> >>>>>
> >>>>
> >>
> >>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dsetrakyan
Looks good, but why in the example provided are we suggesting 8GB? 2 nodes
with 8GB will completely exhaust the available memory. I would suggest 6 or
7GB.

Also, why 100MB for default policy. Anything under 1GB seems too small.

Can you please comment?

D.

On Mon, Aug 28, 2017 at 3:31 AM, Seliverstov Igor <[hidden email]>
wrote:

> One more example of possible warning:
>
> -----------------------------------------------------
> Excessive memory usage by Ignite node process (performance may drop)
> [requested=44613MB, available=15942MB].
>
> Please tune the folowing settings as suggested:
>   MemoryPolicyConfiguration.initialSize for bigPlc: 8102MB
>   MemoryPolicyConfiguration.maxSize     for bigPlc: 8102MB
>   MemoryPolicyConfiguration.initialSize for dfltPlc: 100MB
>   MemoryPolicyConfiguration.maxSize     for dfltPlc: 100MB
>
> Current settings:
>   Java Heap  maxSize: 3543MB
>   Java Heap initSize: 250MB
>   MemoryPolicyConfiguration.initialSize for bigPlc: 256MB
>   MemoryPolicyConfiguration.maxSize     for bigPlc: 40960MB
>   MemoryPolicyConfiguration.initialSize for dfltPlc: 10MB
>   MemoryPolicyConfiguration.maxSize     for dfltPlc: 10MB
>   The overall expected memory usage by all Ignite nodes on the host:
> 44613MB
> -----------------------------------------------------
>
> Your thoughts?
>
> 2017-08-28 5:06 GMT+03:00 Denis Magda <[hidden email]>:
>
> > Guys,
> >
> > ML lib profile is missing in 2.1 release! That must be fixed and rolled
> > out in this emergency release:
> > https://issues.apache.org/jira/browse/IGNITE-6193 <
> > https://issues.apache.org/jira/browse/IGNITE-6193>
> >
> > Oleg, Yuri, please step in and handle the issue.
> >
> > BTW, who is considered to be the release manager of this release?
> >
> > —
> > Denis
> >
> > > On Aug 25, 2017, at 2:29 PM, Dmitriy Setrakyan <[hidden email]>
> > wrote:
> > >
> > > I like the format proposed by Denis, very clear.
> > >
> > > However, I also do not understand why a user should change the size of
> > some
> > > system cache. How would a user ever know what value to put there? This
> > > value should be configured by Ignite automatically.
> > >
> > > D.
> > >
> > > On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <[hidden email]>
> wrote:
> > >
> > >> Igor,
> > >>
> > >> Let me suggest this format.
> > >>
> > >> ---------------------------------------------
> > >> Excessive memory usage by Ignite node process (performance may drop)
> > >> [requested=29251MB, available=15942MB]
> > >>
> > >> Please tune the following settings:
> > >>  [MemoryConfiguration.defaultMemoryPolicySize = suggested value]
> > >>  MemoryConfiguration.systemCacheMaxSize = suggested value
> > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_1} = suggested
> > >> value]
> > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_2} = suggested
> > >> value]
> > >>
> > >> Current settings:
> > >>   [DefaultMemoryPolicySize = value]
> > >>   [{policy_name_1} size = value]
> > >>   [{policy_name_1} size = value]
> > >>   SystemCacheInitialSize = value
> > >>   SystemCacheMaxSize = value
> > >>   Java Heap Init Size = value
> > >>   Java Heap Max Size = value
> > >>
> > >> The overall memory usage by all Ignite nodes on the host: value
> > >> -------------------------------------------
> > >>
> > >> Records in […] are optional. If custom memory policy is not set or the
> > >> default memory policy is overridden the output will miss some of the
> > rows.
> > >>
> > >> As for systemCacheMaxSize, it should be show ONLY if the parameter was
> > set
> > >> explicitly by user code. Otherwise, the platform should be wise enough
> > to
> > >> instantiate it properly depending on the host memory usage.
> > >>
> > >> —
> > >> Denis
> > >>
> > >>> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <[hidden email]>
> > >> wrote:
> > >>>
> > >>> The message without logging layout:
> > >>>
> > >>> Not enough memory for current process [required=29251MB,
> > >> available=15942MB].
> > >>> Please change MemoryConfiguration.systemCacheMaxSize and
> > >>> MemoryConfiguration.defaultMemoryPolicySize to decrease memory
> > allocated
> > >>> for each node.
> > >>>
> > >>> Current settings:
> > >>>  HeapInit=250MB
> > >>>  HeapMax=3543MB
> > >>>  DefaultMemoryPolicySize=12753MB
> > >>>  SystemCacheInitialSize=40MB
> > >>>  SystemCacheMaxSize=100MB
> > >>>
> > >>> Other ignite instanses on the server require: 12853MB
> > >>>
> > >>> I think it's make sense to describe what these numbers are consist
> of.
> > >>>
> > >>> We simple say which parameters have an impact on how much memory the
> > >>> instance needs and their (parameters) actual values.
> > >>>
> > >>> Also we notice that more than one Ignite instance are ran on the
> server
> > >> or
> > >>> workstation and it also consumes memory.
> > >>>
> > >>> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> > >>> [hidden email]> написал:
> > >>>
> > >>>> Igor, what is this flood of WARN messaging coming after the text?
> Are
> > we
> > >>>> really going to print this whole thing out?
> > >>>>
> > >>>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <
> > [hidden email]
> > >>>
> > >>>> wrote:
> > >>>>
> > >>>>> This message appears on topology change in case the available
> memory
> > is
> > >>>>> exceeded
> > >>>>>
> > >>>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <[hidden email]
> >:
> > >>>>>
> > >>>>>> An example of current impl:
> > >>>>>>
> > >>>>>>
> > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory
> for
> > >>>>>> current process [required=29251MB, available=15942MB].
> > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> > >>>>>> MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
> > >>>>> defaultMemoryPolicySize
> > >>>>>> to decrease memory allocated for each node.
> > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
> > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
> > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
> > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >>>> DefaultMemoryPolicySize=
> > >>>>>> 12753MB
> > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >>>>>> SystemCacheInitialSize=40MB
> > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >>>> SystemCacheMaxSize=100MB
> > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite
> > instanses
> > >>>> on
> > >>>>>> the server require: 12853MB
> > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%internal.
> > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >>>>>>
> > >>>>>>
> > >>>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]>:
> > >>>>>>
> > >>>>>>> I suppose we should not forget JVM heap size and suggest to
> reduce
> > >>>> both
> > >>>>>>> options
> > >>>>>>>
> > >>>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> > >>>>> [hidden email]
> > >>>>>>>>
> > >>>>>>> wrote:
> > >>>>>>>
> > >>>>>>>> Igor, I would change the message. How about this:
> > >>>>>>>>
> > >>>>>>>> Required RAM size is larger than total physical memory available
> > for
> > >>>>> OS.
> > >>>>>>>>> Please change MemoryConfiguration.WhichProperty and
> > >>>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease memory
> > >>>>> allocated
> > >>>>>>> for
> > >>>>>>>>> each node.
> > >>>>>>>>
> > >>>>>>>>
> > >>>>>>>> Also, can we calculate what the memory size allocated for each
> > node
> > >>>>>>> should
> > >>>>>>>> be? In that case we should suggest it.
> > >>>>>>>>
> > >>>>>>>> D.
> > >>>>>>>>
> > >>>>>>>>
> > >>>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> > >>>>> [hidden email]
> > >>>>>>>>
> > >>>>>>>> wrote:
> > >>>>>>>>
> > >>>>>>>>> What do you, guys think about next warning?
> > >>>>>>>>>
> > >>>>>>>>> [2017-08-25 17:17:04,718][INFO
> > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >>>>>>> hSelfTest0]
> > >>>>>>>>> System cache's MemoryPolicy size is configured to 40 MB. Use
> > >>>>>>>>> MemoryConfiguration.systemCacheMemorySize property to change
> the
> > >>>>>>>> setting.
> > >>>>>>>>> [2017-08-25 17:17:04,718][WARN
> > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >>>>>>> hSelfTest0]
> > >>>>>>>>>
> > >>>>>>>>>>>> Required RAM size is larger than total physical memory
> > >>>> available
> > >>>>>>> for
> > >>>>>>>>> OS.
> > >>>>>>>>>>>> Check your configuration to avoid swap partition usage.
> > >>>>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration to
> > >>>> change
> > >>>>>>> the
> > >>>>>>>>> settings.
> > >>>>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
> > >>>>>>>>>
> > >>>>>>>>> [2017-08-25 17:17:04,726][WARN
> > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >>>>>>> hSelfTest0]
> > >>>>>>>>> Peer class loading is enabled (disable it in production for
> > >>>>>>> performance
> > >>>>>>>> and
> > >>>>>>>>> deployment consistency reasons)
> > >>>>>>>>> [2017-08-25 17:17:04,726][INFO
> > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >>>>>>> hSelfTest0]
> > >>>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
> > >>>>> ['ignite-sys-cache']]
> > >>>>>>>>> [2017-08-25 17:17:04,731][INFO
> > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > >>>>>>> hSelfTest0]
> > >>>>>>>>> 3-rd party licenses can be found at:
> > >>>>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/
> licenses
> > >>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <[hidden email]
> >:
> > >>>>>>>>>
> > >>>>>>>>>> Agree, let's release new version including tickets mentioned
> by
> > >>>>>>> Denis
> > >>>>>>>> and
> > >>>>>>>>>> Ivan.
> > >>>>>>>>>>
> > >>>>>>>>>> --Yakov
> > >>>>>>>>>>
> > >>>>>>>>>
> > >>>>>>>>
> > >>>>>>>
> > >>>>>>>
> > >>>>>>>
> > >>>>>>> --
> > >>>>>>> Sergey Kozlov
> > >>>>>>> GridGain Systems
> > >>>>>>> www.gridgain.com
> > >>>>>>>
> > >>>>>>
> > >>>>>>
> > >>>>>
> > >>>>
> > >>
> > >>
> >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

gvvinblade
The suggestion here is based on initial settings, and it's so because there
is no other nodes on the host in the example.

The algorithm tries to preserve the original ratio of memory policies
keeping numbers reasonable (for example after some thresshold it will
suggest not to use several memory policies if there is not enough of memory
for all of them) and taking into consideration nodes count on the host,
each jvm heap, needed memory for OS, etc

2017-08-28 14:38 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:

> Looks good, but why in the example provided are we suggesting 8GB? 2 nodes
> with 8GB will completely exhaust the available memory. I would suggest 6 or
> 7GB.
>
> Also, why 100MB for default policy. Anything under 1GB seems too small.
>
> Can you please comment?
>
> D.
>
> On Mon, Aug 28, 2017 at 3:31 AM, Seliverstov Igor <[hidden email]>
> wrote:
>
> > One more example of possible warning:
> >
> > -----------------------------------------------------
> > Excessive memory usage by Ignite node process (performance may drop)
> > [requested=44613MB, available=15942MB].
> >
> > Please tune the folowing settings as suggested:
> >   MemoryPolicyConfiguration.initialSize for bigPlc: 8102MB
> >   MemoryPolicyConfiguration.maxSize     for bigPlc: 8102MB
> >   MemoryPolicyConfiguration.initialSize for dfltPlc: 100MB
> >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 100MB
> >
> > Current settings:
> >   Java Heap  maxSize: 3543MB
> >   Java Heap initSize: 250MB
> >   MemoryPolicyConfiguration.initialSize for bigPlc: 256MB
> >   MemoryPolicyConfiguration.maxSize     for bigPlc: 40960MB
> >   MemoryPolicyConfiguration.initialSize for dfltPlc: 10MB
> >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 10MB
> >   The overall expected memory usage by all Ignite nodes on the host:
> > 44613MB
> > -----------------------------------------------------
> >
> > Your thoughts?
> >
> > 2017-08-28 5:06 GMT+03:00 Denis Magda <[hidden email]>:
> >
> > > Guys,
> > >
> > > ML lib profile is missing in 2.1 release! That must be fixed and rolled
> > > out in this emergency release:
> > > https://issues.apache.org/jira/browse/IGNITE-6193 <
> > > https://issues.apache.org/jira/browse/IGNITE-6193>
> > >
> > > Oleg, Yuri, please step in and handle the issue.
> > >
> > > BTW, who is considered to be the release manager of this release?
> > >
> > > —
> > > Denis
> > >
> > > > On Aug 25, 2017, at 2:29 PM, Dmitriy Setrakyan <
> [hidden email]>
> > > wrote:
> > > >
> > > > I like the format proposed by Denis, very clear.
> > > >
> > > > However, I also do not understand why a user should change the size
> of
> > > some
> > > > system cache. How would a user ever know what value to put there?
> This
> > > > value should be configured by Ignite automatically.
> > > >
> > > > D.
> > > >
> > > > On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <[hidden email]>
> > wrote:
> > > >
> > > >> Igor,
> > > >>
> > > >> Let me suggest this format.
> > > >>
> > > >> ---------------------------------------------
> > > >> Excessive memory usage by Ignite node process (performance may drop)
> > > >> [requested=29251MB, available=15942MB]
> > > >>
> > > >> Please tune the following settings:
> > > >>  [MemoryConfiguration.defaultMemoryPolicySize = suggested value]
> > > >>  MemoryConfiguration.systemCacheMaxSize = suggested value
> > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_1} = suggested
> > > >> value]
> > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_2} = suggested
> > > >> value]
> > > >>
> > > >> Current settings:
> > > >>   [DefaultMemoryPolicySize = value]
> > > >>   [{policy_name_1} size = value]
> > > >>   [{policy_name_1} size = value]
> > > >>   SystemCacheInitialSize = value
> > > >>   SystemCacheMaxSize = value
> > > >>   Java Heap Init Size = value
> > > >>   Java Heap Max Size = value
> > > >>
> > > >> The overall memory usage by all Ignite nodes on the host: value
> > > >> -------------------------------------------
> > > >>
> > > >> Records in […] are optional. If custom memory policy is not set or
> the
> > > >> default memory policy is overridden the output will miss some of the
> > > rows.
> > > >>
> > > >> As for systemCacheMaxSize, it should be show ONLY if the parameter
> was
> > > set
> > > >> explicitly by user code. Otherwise, the platform should be wise
> enough
> > > to
> > > >> instantiate it properly depending on the host memory usage.
> > > >>
> > > >> —
> > > >> Denis
> > > >>
> > > >>> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <
> [hidden email]>
> > > >> wrote:
> > > >>>
> > > >>> The message without logging layout:
> > > >>>
> > > >>> Not enough memory for current process [required=29251MB,
> > > >> available=15942MB].
> > > >>> Please change MemoryConfiguration.systemCacheMaxSize and
> > > >>> MemoryConfiguration.defaultMemoryPolicySize to decrease memory
> > > allocated
> > > >>> for each node.
> > > >>>
> > > >>> Current settings:
> > > >>>  HeapInit=250MB
> > > >>>  HeapMax=3543MB
> > > >>>  DefaultMemoryPolicySize=12753MB
> > > >>>  SystemCacheInitialSize=40MB
> > > >>>  SystemCacheMaxSize=100MB
> > > >>>
> > > >>> Other ignite instanses on the server require: 12853MB
> > > >>>
> > > >>> I think it's make sense to describe what these numbers are consist
> > of.
> > > >>>
> > > >>> We simple say which parameters have an impact on how much memory
> the
> > > >>> instance needs and their (parameters) actual values.
> > > >>>
> > > >>> Also we notice that more than one Ignite instance are ran on the
> > server
> > > >> or
> > > >>> workstation and it also consumes memory.
> > > >>>
> > > >>> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> > > >>> [hidden email]> написал:
> > > >>>
> > > >>>> Igor, what is this flood of WARN messaging coming after the text?
> > Are
> > > we
> > > >>>> really going to print this whole thing out?
> > > >>>>
> > > >>>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <
> > > [hidden email]
> > > >>>
> > > >>>> wrote:
> > > >>>>
> > > >>>>> This message appears on topology change in case the available
> > memory
> > > is
> > > >>>>> exceeded
> > > >>>>>
> > > >>>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <
> [hidden email]
> > >:
> > > >>>>>
> > > >>>>>> An example of current impl:
> > > >>>>>>
> > > >>>>>>
> > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough memory
> > for
> > > >>>>>> current process [required=29251MB, available=15942MB].
> > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> > > >>>>>> MemoryConfiguration.systemCacheMaxSize and MemoryConfiguration.
> > > >>>>> defaultMemoryPolicySize
> > > >>>>>> to decrease memory allocated for each node.
> > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current settings:
> > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapInit=250MB
> > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]   HeapMax=3543MB
> > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > >>>> DefaultMemoryPolicySize=
> > > >>>>>> 12753MB
> > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > >>>>>> SystemCacheInitialSize=40MB
> > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > >>>> SystemCacheMaxSize=100MB
> > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite
> > > instanses
> > > >>>> on
> > > >>>>>> the server require: 12853MB
> > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> internal.
> > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > >>>>>>
> > > >>>>>>
> > > >>>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <[hidden email]
> >:
> > > >>>>>>
> > > >>>>>>> I suppose we should not forget JVM heap size and suggest to
> > reduce
> > > >>>> both
> > > >>>>>>> options
> > > >>>>>>>
> > > >>>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> > > >>>>> [hidden email]
> > > >>>>>>>>
> > > >>>>>>> wrote:
> > > >>>>>>>
> > > >>>>>>>> Igor, I would change the message. How about this:
> > > >>>>>>>>
> > > >>>>>>>> Required RAM size is larger than total physical memory
> available
> > > for
> > > >>>>> OS.
> > > >>>>>>>>> Please change MemoryConfiguration.WhichProperty and
> > > >>>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease memory
> > > >>>>> allocated
> > > >>>>>>> for
> > > >>>>>>>>> each node.
> > > >>>>>>>>
> > > >>>>>>>>
> > > >>>>>>>> Also, can we calculate what the memory size allocated for each
> > > node
> > > >>>>>>> should
> > > >>>>>>>> be? In that case we should suggest it.
> > > >>>>>>>>
> > > >>>>>>>> D.
> > > >>>>>>>>
> > > >>>>>>>>
> > > >>>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> > > >>>>> [hidden email]
> > > >>>>>>>>
> > > >>>>>>>> wrote:
> > > >>>>>>>>
> > > >>>>>>>>> What do you, guys think about next warning?
> > > >>>>>>>>>
> > > >>>>>>>>> [2017-08-25 17:17:04,718][INFO
> > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > > >>>>>>> hSelfTest0]
> > > >>>>>>>>> System cache's MemoryPolicy size is configured to 40 MB. Use
> > > >>>>>>>>> MemoryConfiguration.systemCacheMemorySize property to change
> > the
> > > >>>>>>>> setting.
> > > >>>>>>>>> [2017-08-25 17:17:04,718][WARN
> > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > > >>>>>>> hSelfTest0]
> > > >>>>>>>>>
> > > >>>>>>>>>>>> Required RAM size is larger than total physical memory
> > > >>>> available
> > > >>>>>>> for
> > > >>>>>>>>> OS.
> > > >>>>>>>>>>>> Check your configuration to avoid swap partition usage.
> > > >>>>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration to
> > > >>>> change
> > > >>>>>>> the
> > > >>>>>>>>> settings.
> > > >>>>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
> > > >>>>>>>>>
> > > >>>>>>>>> [2017-08-25 17:17:04,726][WARN
> > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > > >>>>>>> hSelfTest0]
> > > >>>>>>>>> Peer class loading is enabled (disable it in production for
> > > >>>>>>> performance
> > > >>>>>>>> and
> > > >>>>>>>>> deployment consistency reasons)
> > > >>>>>>>>> [2017-08-25 17:17:04,726][INFO
> > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > > >>>>>>> hSelfTest0]
> > > >>>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
> > > >>>>> ['ignite-sys-cache']]
> > > >>>>>>>>> [2017-08-25 17:17:04,731][INFO
> > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][GridHomePat
> > > >>>>>>> hSelfTest0]
> > > >>>>>>>>> 3-rd party licenses can be found at:
> > > >>>>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/
> > licenses
> > > >>>>>>>>>
> > > >>>>>>>>>
> > > >>>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <
> [hidden email]
> > >:
> > > >>>>>>>>>
> > > >>>>>>>>>> Agree, let's release new version including tickets mentioned
> > by
> > > >>>>>>> Denis
> > > >>>>>>>> and
> > > >>>>>>>>>> Ivan.
> > > >>>>>>>>>>
> > > >>>>>>>>>> --Yakov
> > > >>>>>>>>>>
> > > >>>>>>>>>
> > > >>>>>>>>
> > > >>>>>>>
> > > >>>>>>>
> > > >>>>>>>
> > > >>>>>>> --
> > > >>>>>>> Sergey Kozlov
> > > >>>>>>> GridGain Systems
> > > >>>>>>> www.gridgain.com
> > > >>>>>>>
> > > >>>>>>
> > > >>>>>>
> > > >>>>>
> > > >>>>
> > > >>
> > > >>
> > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

dsetrakyan
Igor, can you please describe the algorithm with all the thresholds?

On Mon, Aug 28, 2017 at 4:56 AM, Seliverstov Igor <[hidden email]>
wrote:

> The suggestion here is based on initial settings, and it's so because there
> is no other nodes on the host in the example.
>
> The algorithm tries to preserve the original ratio of memory policies
> keeping numbers reasonable (for example after some thresshold it will
> suggest not to use several memory policies if there is not enough of memory
> for all of them) and taking into consideration nodes count on the host,
> each jvm heap, needed memory for OS, etc
>
> 2017-08-28 14:38 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:
>
> > Looks good, but why in the example provided are we suggesting 8GB? 2
> nodes
> > with 8GB will completely exhaust the available memory. I would suggest 6
> or
> > 7GB.
> >
> > Also, why 100MB for default policy. Anything under 1GB seems too small.
> >
> > Can you please comment?
> >
> > D.
> >
> > On Mon, Aug 28, 2017 at 3:31 AM, Seliverstov Igor <[hidden email]>
> > wrote:
> >
> > > One more example of possible warning:
> > >
> > > -----------------------------------------------------
> > > Excessive memory usage by Ignite node process (performance may drop)
> > > [requested=44613MB, available=15942MB].
> > >
> > > Please tune the folowing settings as suggested:
> > >   MemoryPolicyConfiguration.initialSize for bigPlc: 8102MB
> > >   MemoryPolicyConfiguration.maxSize     for bigPlc: 8102MB
> > >   MemoryPolicyConfiguration.initialSize for dfltPlc: 100MB
> > >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 100MB
> > >
> > > Current settings:
> > >   Java Heap  maxSize: 3543MB
> > >   Java Heap initSize: 250MB
> > >   MemoryPolicyConfiguration.initialSize for bigPlc: 256MB
> > >   MemoryPolicyConfiguration.maxSize     for bigPlc: 40960MB
> > >   MemoryPolicyConfiguration.initialSize for dfltPlc: 10MB
> > >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 10MB
> > >   The overall expected memory usage by all Ignite nodes on the host:
> > > 44613MB
> > > -----------------------------------------------------
> > >
> > > Your thoughts?
> > >
> > > 2017-08-28 5:06 GMT+03:00 Denis Magda <[hidden email]>:
> > >
> > > > Guys,
> > > >
> > > > ML lib profile is missing in 2.1 release! That must be fixed and
> rolled
> > > > out in this emergency release:
> > > > https://issues.apache.org/jira/browse/IGNITE-6193 <
> > > > https://issues.apache.org/jira/browse/IGNITE-6193>
> > > >
> > > > Oleg, Yuri, please step in and handle the issue.
> > > >
> > > > BTW, who is considered to be the release manager of this release?
> > > >
> > > > —
> > > > Denis
> > > >
> > > > > On Aug 25, 2017, at 2:29 PM, Dmitriy Setrakyan <
> > [hidden email]>
> > > > wrote:
> > > > >
> > > > > I like the format proposed by Denis, very clear.
> > > > >
> > > > > However, I also do not understand why a user should change the size
> > of
> > > > some
> > > > > system cache. How would a user ever know what value to put there?
> > This
> > > > > value should be configured by Ignite automatically.
> > > > >
> > > > > D.
> > > > >
> > > > > On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <[hidden email]>
> > > wrote:
> > > > >
> > > > >> Igor,
> > > > >>
> > > > >> Let me suggest this format.
> > > > >>
> > > > >> ---------------------------------------------
> > > > >> Excessive memory usage by Ignite node process (performance may
> drop)
> > > > >> [requested=29251MB, available=15942MB]
> > > > >>
> > > > >> Please tune the following settings:
> > > > >>  [MemoryConfiguration.defaultMemoryPolicySize = suggested value]
> > > > >>  MemoryConfiguration.systemCacheMaxSize = suggested value
> > > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_1} =
> suggested
> > > > >> value]
> > > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_2} =
> suggested
> > > > >> value]
> > > > >>
> > > > >> Current settings:
> > > > >>   [DefaultMemoryPolicySize = value]
> > > > >>   [{policy_name_1} size = value]
> > > > >>   [{policy_name_1} size = value]
> > > > >>   SystemCacheInitialSize = value
> > > > >>   SystemCacheMaxSize = value
> > > > >>   Java Heap Init Size = value
> > > > >>   Java Heap Max Size = value
> > > > >>
> > > > >> The overall memory usage by all Ignite nodes on the host: value
> > > > >> -------------------------------------------
> > > > >>
> > > > >> Records in […] are optional. If custom memory policy is not set or
> > the
> > > > >> default memory policy is overridden the output will miss some of
> the
> > > > rows.
> > > > >>
> > > > >> As for systemCacheMaxSize, it should be show ONLY if the parameter
> > was
> > > > set
> > > > >> explicitly by user code. Otherwise, the platform should be wise
> > enough
> > > > to
> > > > >> instantiate it properly depending on the host memory usage.
> > > > >>
> > > > >> —
> > > > >> Denis
> > > > >>
> > > > >>> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <
> > [hidden email]>
> > > > >> wrote:
> > > > >>>
> > > > >>> The message without logging layout:
> > > > >>>
> > > > >>> Not enough memory for current process [required=29251MB,
> > > > >> available=15942MB].
> > > > >>> Please change MemoryConfiguration.systemCacheMaxSize and
> > > > >>> MemoryConfiguration.defaultMemoryPolicySize to decrease memory
> > > > allocated
> > > > >>> for each node.
> > > > >>>
> > > > >>> Current settings:
> > > > >>>  HeapInit=250MB
> > > > >>>  HeapMax=3543MB
> > > > >>>  DefaultMemoryPolicySize=12753MB
> > > > >>>  SystemCacheInitialSize=40MB
> > > > >>>  SystemCacheMaxSize=100MB
> > > > >>>
> > > > >>> Other ignite instanses on the server require: 12853MB
> > > > >>>
> > > > >>> I think it's make sense to describe what these numbers are
> consist
> > > of.
> > > > >>>
> > > > >>> We simple say which parameters have an impact on how much memory
> > the
> > > > >>> instance needs and their (parameters) actual values.
> > > > >>>
> > > > >>> Also we notice that more than one Ignite instance are ran on the
> > > server
> > > > >> or
> > > > >>> workstation and it also consumes memory.
> > > > >>>
> > > > >>> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> > > > >>> [hidden email]> написал:
> > > > >>>
> > > > >>>> Igor, what is this flood of WARN messaging coming after the
> text?
> > > Are
> > > > we
> > > > >>>> really going to print this whole thing out?
> > > > >>>>
> > > > >>>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <
> > > > [hidden email]
> > > > >>>
> > > > >>>> wrote:
> > > > >>>>
> > > > >>>>> This message appears on topology change in case the available
> > > memory
> > > > is
> > > > >>>>> exceeded
> > > > >>>>>
> > > > >>>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <
> > [hidden email]
> > > >:
> > > > >>>>>
> > > > >>>>>> An example of current impl:
> > > > >>>>>>
> > > > >>>>>>
> > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough
> memory
> > > for
> > > > >>>>>> current process [required=29251MB, available=15942MB].
> > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> > > > >>>>>> MemoryConfiguration.systemCacheMaxSize and
> MemoryConfiguration.
> > > > >>>>> defaultMemoryPolicySize
> > > > >>>>>> to decrease memory allocated for each node.
> > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current
> settings:
> > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>  HeapInit=250MB
> > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
>  HeapMax=3543MB
> > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > >>>> DefaultMemoryPolicySize=
> > > > >>>>>> 12753MB
> > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > >>>>>> SystemCacheInitialSize=40MB
> > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > >>>> SystemCacheMaxSize=100MB
> > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite
> > > > instanses
> > > > >>>> on
> > > > >>>>>> the server require: 12853MB
> > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > internal.
> > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > >>>>>>
> > > > >>>>>>
> > > > >>>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <
> [hidden email]
> > >:
> > > > >>>>>>
> > > > >>>>>>> I suppose we should not forget JVM heap size and suggest to
> > > reduce
> > > > >>>> both
> > > > >>>>>>> options
> > > > >>>>>>>
> > > > >>>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> > > > >>>>> [hidden email]
> > > > >>>>>>>>
> > > > >>>>>>> wrote:
> > > > >>>>>>>
> > > > >>>>>>>> Igor, I would change the message. How about this:
> > > > >>>>>>>>
> > > > >>>>>>>> Required RAM size is larger than total physical memory
> > available
> > > > for
> > > > >>>>> OS.
> > > > >>>>>>>>> Please change MemoryConfiguration.WhichProperty and
> > > > >>>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease memory
> > > > >>>>> allocated
> > > > >>>>>>> for
> > > > >>>>>>>>> each node.
> > > > >>>>>>>>
> > > > >>>>>>>>
> > > > >>>>>>>> Also, can we calculate what the memory size allocated for
> each
> > > > node
> > > > >>>>>>> should
> > > > >>>>>>>> be? In that case we should suggest it.
> > > > >>>>>>>>
> > > > >>>>>>>> D.
> > > > >>>>>>>>
> > > > >>>>>>>>
> > > > >>>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> > > > >>>>> [hidden email]
> > > > >>>>>>>>
> > > > >>>>>>>> wrote:
> > > > >>>>>>>>
> > > > >>>>>>>>> What do you, guys think about next warning?
> > > > >>>>>>>>>
> > > > >>>>>>>>> [2017-08-25 17:17:04,718][INFO
> > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePat
> > > > >>>>>>> hSelfTest0]
> > > > >>>>>>>>> System cache's MemoryPolicy size is configured to 40 MB.
> Use
> > > > >>>>>>>>> MemoryConfiguration.systemCacheMemorySize property to
> change
> > > the
> > > > >>>>>>>> setting.
> > > > >>>>>>>>> [2017-08-25 17:17:04,718][WARN
> > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePat
> > > > >>>>>>> hSelfTest0]
> > > > >>>>>>>>>
> > > > >>>>>>>>>>>> Required RAM size is larger than total physical memory
> > > > >>>> available
> > > > >>>>>>> for
> > > > >>>>>>>>> OS.
> > > > >>>>>>>>>>>> Check your configuration to avoid swap partition usage.
> > > > >>>>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration to
> > > > >>>> change
> > > > >>>>>>> the
> > > > >>>>>>>>> settings.
> > > > >>>>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
> > > > >>>>>>>>>
> > > > >>>>>>>>> [2017-08-25 17:17:04,726][WARN
> > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePat
> > > > >>>>>>> hSelfTest0]
> > > > >>>>>>>>> Peer class loading is enabled (disable it in production for
> > > > >>>>>>> performance
> > > > >>>>>>>> and
> > > > >>>>>>>>> deployment consistency reasons)
> > > > >>>>>>>>> [2017-08-25 17:17:04,726][INFO
> > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePat
> > > > >>>>>>> hSelfTest0]
> > > > >>>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
> > > > >>>>> ['ignite-sys-cache']]
> > > > >>>>>>>>> [2017-08-25 17:17:04,731][INFO
> > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> GridHomePat
> > > > >>>>>>> hSelfTest0]
> > > > >>>>>>>>> 3-rd party licenses can be found at:
> > > > >>>>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/
> > > licenses
> > > > >>>>>>>>>
> > > > >>>>>>>>>
> > > > >>>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <
> > [hidden email]
> > > >:
> > > > >>>>>>>>>
> > > > >>>>>>>>>> Agree, let's release new version including tickets
> mentioned
> > > by
> > > > >>>>>>> Denis
> > > > >>>>>>>> and
> > > > >>>>>>>>>> Ivan.
> > > > >>>>>>>>>>
> > > > >>>>>>>>>> --Yakov
> > > > >>>>>>>>>>
> > > > >>>>>>>>>
> > > > >>>>>>>>
> > > > >>>>>>>
> > > > >>>>>>>
> > > > >>>>>>>
> > > > >>>>>>> --
> > > > >>>>>>> Sergey Kozlov
> > > > >>>>>>> GridGain Systems
> > > > >>>>>>> www.gridgain.com
> > > > >>>>>>>
> > > > >>>>>>
> > > > >>>>>>
> > > > >>>>>
> > > > >>>>
> > > > >>
> > > > >>
> > > >
> > > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

gvvinblade
Ok, the check happens at the node start time or on NODE_JOIN event

in general it looks like:

1) calculate expected used memory = heap max + system cache max + all
custom policies max + default policy size and put it into a node attribute

2) get total physycal memory, calculate expected safe to be used memory
amount (leave 4 gb min or 20% of available memory for OS)

3) if expected used memory + expected used memory of other nodes on the
host > than safe to be used memory amount, start calculating suggestions

4) Each ignite instance needs at least 512mb heap + 40mb system cache +
100mb default polycy, if available memory is less we cannot suggest
anything reasonable, print warning, stop calculation.

5) check heap size (shouldn't exceed 30% of available memory (total_memory
- reserved for OS memory) * 30% for all JVMs, if it exeedes, suggest just
calculated value or 512MB minimal)

6) check if system cache size changed, suggest default value if it's so

7) in case 100 mb * policies count < available memory, suggest using
default policy with max size equals to remaining memory (available - heap -
system cache)

8) calculate new size for each memory policy ( it's user defined size *
(remaining / (all_policies_size * nodes_cnt)); in proportion to
remaining memory, devided by nodes number on the host or 100 mb minimal)

9) print suggestions



2017-08-28 15:10 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:

> Igor, can you please describe the algorithm with all the thresholds?
>
> On Mon, Aug 28, 2017 at 4:56 AM, Seliverstov Igor <[hidden email]>
> wrote:
>
> > The suggestion here is based on initial settings, and it's so because
> there
> > is no other nodes on the host in the example.
> >
> > The algorithm tries to preserve the original ratio of memory policies
> > keeping numbers reasonable (for example after some thresshold it will
> > suggest not to use several memory policies if there is not enough of
> memory
> > for all of them) and taking into consideration nodes count on the host,
> > each jvm heap, needed memory for OS, etc
> >
> > 2017-08-28 14:38 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:
> >
> > > Looks good, but why in the example provided are we suggesting 8GB? 2
> > nodes
> > > with 8GB will completely exhaust the available memory. I would suggest
> 6
> > or
> > > 7GB.
> > >
> > > Also, why 100MB for default policy. Anything under 1GB seems too small.
> > >
> > > Can you please comment?
> > >
> > > D.
> > >
> > > On Mon, Aug 28, 2017 at 3:31 AM, Seliverstov Igor <
> [hidden email]>
> > > wrote:
> > >
> > > > One more example of possible warning:
> > > >
> > > > -----------------------------------------------------
> > > > Excessive memory usage by Ignite node process (performance may drop)
> > > > [requested=44613MB, available=15942MB].
> > > >
> > > > Please tune the folowing settings as suggested:
> > > >   MemoryPolicyConfiguration.initialSize for bigPlc: 8102MB
> > > >   MemoryPolicyConfiguration.maxSize     for bigPlc: 8102MB
> > > >   MemoryPolicyConfiguration.initialSize for dfltPlc: 100MB
> > > >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 100MB
> > > >
> > > > Current settings:
> > > >   Java Heap  maxSize: 3543MB
> > > >   Java Heap initSize: 250MB
> > > >   MemoryPolicyConfiguration.initialSize for bigPlc: 256MB
> > > >   MemoryPolicyConfiguration.maxSize     for bigPlc: 40960MB
> > > >   MemoryPolicyConfiguration.initialSize for dfltPlc: 10MB
> > > >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 10MB
> > > >   The overall expected memory usage by all Ignite nodes on the host:
> > > > 44613MB
> > > > -----------------------------------------------------
> > > >
> > > > Your thoughts?
> > > >
> > > > 2017-08-28 5:06 GMT+03:00 Denis Magda <[hidden email]>:
> > > >
> > > > > Guys,
> > > > >
> > > > > ML lib profile is missing in 2.1 release! That must be fixed and
> > rolled
> > > > > out in this emergency release:
> > > > > https://issues.apache.org/jira/browse/IGNITE-6193 <
> > > > > https://issues.apache.org/jira/browse/IGNITE-6193>
> > > > >
> > > > > Oleg, Yuri, please step in and handle the issue.
> > > > >
> > > > > BTW, who is considered to be the release manager of this release?
> > > > >
> > > > > —
> > > > > Denis
> > > > >
> > > > > > On Aug 25, 2017, at 2:29 PM, Dmitriy Setrakyan <
> > > [hidden email]>
> > > > > wrote:
> > > > > >
> > > > > > I like the format proposed by Denis, very clear.
> > > > > >
> > > > > > However, I also do not understand why a user should change the
> size
> > > of
> > > > > some
> > > > > > system cache. How would a user ever know what value to put there?
> > > This
> > > > > > value should be configured by Ignite automatically.
> > > > > >
> > > > > > D.
> > > > > >
> > > > > > On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <[hidden email]>
> > > > wrote:
> > > > > >
> > > > > >> Igor,
> > > > > >>
> > > > > >> Let me suggest this format.
> > > > > >>
> > > > > >> ---------------------------------------------
> > > > > >> Excessive memory usage by Ignite node process (performance may
> > drop)
> > > > > >> [requested=29251MB, available=15942MB]
> > > > > >>
> > > > > >> Please tune the following settings:
> > > > > >>  [MemoryConfiguration.defaultMemoryPolicySize = suggested
> value]
> > > > > >>  MemoryConfiguration.systemCacheMaxSize = suggested value
> > > > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_1} =
> > suggested
> > > > > >> value]
> > > > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_2} =
> > suggested
> > > > > >> value]
> > > > > >>
> > > > > >> Current settings:
> > > > > >>   [DefaultMemoryPolicySize = value]
> > > > > >>   [{policy_name_1} size = value]
> > > > > >>   [{policy_name_1} size = value]
> > > > > >>   SystemCacheInitialSize = value
> > > > > >>   SystemCacheMaxSize = value
> > > > > >>   Java Heap Init Size = value
> > > > > >>   Java Heap Max Size = value
> > > > > >>
> > > > > >> The overall memory usage by all Ignite nodes on the host: value
> > > > > >> -------------------------------------------
> > > > > >>
> > > > > >> Records in […] are optional. If custom memory policy is not set
> or
> > > the
> > > > > >> default memory policy is overridden the output will miss some of
> > the
> > > > > rows.
> > > > > >>
> > > > > >> As for systemCacheMaxSize, it should be show ONLY if the
> parameter
> > > was
> > > > > set
> > > > > >> explicitly by user code. Otherwise, the platform should be wise
> > > enough
> > > > > to
> > > > > >> instantiate it properly depending on the host memory usage.
> > > > > >>
> > > > > >> —
> > > > > >> Denis
> > > > > >>
> > > > > >>> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <
> > > [hidden email]>
> > > > > >> wrote:
> > > > > >>>
> > > > > >>> The message without logging layout:
> > > > > >>>
> > > > > >>> Not enough memory for current process [required=29251MB,
> > > > > >> available=15942MB].
> > > > > >>> Please change MemoryConfiguration.systemCacheMaxSize and
> > > > > >>> MemoryConfiguration.defaultMemoryPolicySize to decrease memory
> > > > > allocated
> > > > > >>> for each node.
> > > > > >>>
> > > > > >>> Current settings:
> > > > > >>>  HeapInit=250MB
> > > > > >>>  HeapMax=3543MB
> > > > > >>>  DefaultMemoryPolicySize=12753MB
> > > > > >>>  SystemCacheInitialSize=40MB
> > > > > >>>  SystemCacheMaxSize=100MB
> > > > > >>>
> > > > > >>> Other ignite instanses on the server require: 12853MB
> > > > > >>>
> > > > > >>> I think it's make sense to describe what these numbers are
> > consist
> > > > of.
> > > > > >>>
> > > > > >>> We simple say which parameters have an impact on how much
> memory
> > > the
> > > > > >>> instance needs and their (parameters) actual values.
> > > > > >>>
> > > > > >>> Also we notice that more than one Ignite instance are ran on
> the
> > > > server
> > > > > >> or
> > > > > >>> workstation and it also consumes memory.
> > > > > >>>
> > > > > >>> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> > > > > >>> [hidden email]> написал:
> > > > > >>>
> > > > > >>>> Igor, what is this flood of WARN messaging coming after the
> > text?
> > > > Are
> > > > > we
> > > > > >>>> really going to print this whole thing out?
> > > > > >>>>
> > > > > >>>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <
> > > > > [hidden email]
> > > > > >>>
> > > > > >>>> wrote:
> > > > > >>>>
> > > > > >>>>> This message appears on topology change in case the available
> > > > memory
> > > > > is
> > > > > >>>>> exceeded
> > > > > >>>>>
> > > > > >>>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <
> > > [hidden email]
> > > > >:
> > > > > >>>>>
> > > > > >>>>>> An example of current impl:
> > > > > >>>>>>
> > > > > >>>>>>
> > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough
> > memory
> > > > for
> > > > > >>>>>> current process [required=29251MB, available=15942MB].
> > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please change
> > > > > >>>>>> MemoryConfiguration.systemCacheMaxSize and
> > MemoryConfiguration.
> > > > > >>>>> defaultMemoryPolicySize
> > > > > >>>>>> to decrease memory allocated for each node.
> > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current
> > settings:
> > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >  HeapInit=250MB
> > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> >  HeapMax=3543MB
> > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > >>>> DefaultMemoryPolicySize=
> > > > > >>>>>> 12753MB
> > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > >>>>>> SystemCacheInitialSize=40MB
> > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > >>>> SystemCacheMaxSize=100MB
> > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other ignite
> > > > > instanses
> > > > > >>>> on
> > > > > >>>>>> the server require: 12853MB
> > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > internal.
> > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > >>>>>>
> > > > > >>>>>>
> > > > > >>>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <
> > [hidden email]
> > > >:
> > > > > >>>>>>
> > > > > >>>>>>> I suppose we should not forget JVM heap size and suggest to
> > > > reduce
> > > > > >>>> both
> > > > > >>>>>>> options
> > > > > >>>>>>>
> > > > > >>>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> > > > > >>>>> [hidden email]
> > > > > >>>>>>>>
> > > > > >>>>>>> wrote:
> > > > > >>>>>>>
> > > > > >>>>>>>> Igor, I would change the message. How about this:
> > > > > >>>>>>>>
> > > > > >>>>>>>> Required RAM size is larger than total physical memory
> > > available
> > > > > for
> > > > > >>>>> OS.
> > > > > >>>>>>>>> Please change MemoryConfiguration.WhichProperty and
> > > > > >>>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease
> memory
> > > > > >>>>> allocated
> > > > > >>>>>>> for
> > > > > >>>>>>>>> each node.
> > > > > >>>>>>>>
> > > > > >>>>>>>>
> > > > > >>>>>>>> Also, can we calculate what the memory size allocated for
> > each
> > > > > node
> > > > > >>>>>>> should
> > > > > >>>>>>>> be? In that case we should suggest it.
> > > > > >>>>>>>>
> > > > > >>>>>>>> D.
> > > > > >>>>>>>>
> > > > > >>>>>>>>
> > > > > >>>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> > > > > >>>>> [hidden email]
> > > > > >>>>>>>>
> > > > > >>>>>>>> wrote:
> > > > > >>>>>>>>
> > > > > >>>>>>>>> What do you, guys think about next warning?
> > > > > >>>>>>>>>
> > > > > >>>>>>>>> [2017-08-25 17:17:04,718][INFO
> > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > GridHomePat
> > > > > >>>>>>> hSelfTest0]
> > > > > >>>>>>>>> System cache's MemoryPolicy size is configured to 40 MB.
> > Use
> > > > > >>>>>>>>> MemoryConfiguration.systemCacheMemorySize property to
> > change
> > > > the
> > > > > >>>>>>>> setting.
> > > > > >>>>>>>>> [2017-08-25 17:17:04,718][WARN
> > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > GridHomePat
> > > > > >>>>>>> hSelfTest0]
> > > > > >>>>>>>>>
> > > > > >>>>>>>>>>>> Required RAM size is larger than total physical memory
> > > > > >>>> available
> > > > > >>>>>>> for
> > > > > >>>>>>>>> OS.
> > > > > >>>>>>>>>>>> Check your configuration to avoid swap partition
> usage.
> > > > > >>>>>>>>>>>> Use MemoryConfiguration and MemoryPolicyConfiguration
> to
> > > > > >>>> change
> > > > > >>>>>>> the
> > > > > >>>>>>>>> settings.
> > > > > >>>>>>>>>>>> Physical memory [required=16397MB, available=15942MB]
> > > > > >>>>>>>>>
> > > > > >>>>>>>>> [2017-08-25 17:17:04,726][WARN
> > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > GridHomePat
> > > > > >>>>>>> hSelfTest0]
> > > > > >>>>>>>>> Peer class loading is enabled (disable it in production
> for
> > > > > >>>>>>> performance
> > > > > >>>>>>>> and
> > > > > >>>>>>>>> deployment consistency reasons)
> > > > > >>>>>>>>> [2017-08-25 17:17:04,726][INFO
> > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > GridHomePat
> > > > > >>>>>>> hSelfTest0]
> > > > > >>>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
> > > > > >>>>> ['ignite-sys-cache']]
> > > > > >>>>>>>>> [2017-08-25 17:17:04,731][INFO
> > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > GridHomePat
> > > > > >>>>>>> hSelfTest0]
> > > > > >>>>>>>>> 3-rd party licenses can be found at:
> > > > > >>>>>>>>> /home/gvvinblade/projects/ignite/incubator-ignite/libs/
> > > > licenses
> > > > > >>>>>>>>>
> > > > > >>>>>>>>>
> > > > > >>>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <
> > > [hidden email]
> > > > >:
> > > > > >>>>>>>>>
> > > > > >>>>>>>>>> Agree, let's release new version including tickets
> > mentioned
> > > > by
> > > > > >>>>>>> Denis
> > > > > >>>>>>>> and
> > > > > >>>>>>>>>> Ivan.
> > > > > >>>>>>>>>>
> > > > > >>>>>>>>>> --Yakov
> > > > > >>>>>>>>>>
> > > > > >>>>>>>>>
> > > > > >>>>>>>>
> > > > > >>>>>>>
> > > > > >>>>>>>
> > > > > >>>>>>>
> > > > > >>>>>>> --
> > > > > >>>>>>> Sergey Kozlov
> > > > > >>>>>>> GridGain Systems
> > > > > >>>>>>> www.gridgain.com
> > > > > >>>>>>>
> > > > > >>>>>>
> > > > > >>>>>>
> > > > > >>>>>
> > > > > >>>>
> > > > > >>
> > > > > >>
> > > > >
> > > > >
> > > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSSION] Urgent Ignite bug fix release

Anton Vinogradov
Denis,

> BTW, who is considered to be the release manager of this release?
I'll do it.

On Mon, Aug 28, 2017 at 3:54 PM, Seliverstov Igor <[hidden email]>
wrote:

> Ok, the check happens at the node start time or on NODE_JOIN event
>
> in general it looks like:
>
> 1) calculate expected used memory = heap max + system cache max + all
> custom policies max + default policy size and put it into a node attribute
>
> 2) get total physycal memory, calculate expected safe to be used memory
> amount (leave 4 gb min or 20% of available memory for OS)
>
> 3) if expected used memory + expected used memory of other nodes on the
> host > than safe to be used memory amount, start calculating suggestions
>
> 4) Each ignite instance needs at least 512mb heap + 40mb system cache +
> 100mb default polycy, if available memory is less we cannot suggest
> anything reasonable, print warning, stop calculation.
>
> 5) check heap size (shouldn't exceed 30% of available memory (total_memory
> - reserved for OS memory) * 30% for all JVMs, if it exeedes, suggest just
> calculated value or 512MB minimal)
>
> 6) check if system cache size changed, suggest default value if it's so
>
> 7) in case 100 mb * policies count < available memory, suggest using
> default policy with max size equals to remaining memory (available - heap -
> system cache)
>
> 8) calculate new size for each memory policy ( it's user defined size *
> (remaining / (all_policies_size * nodes_cnt)); in proportion to
> remaining memory, devided by nodes number on the host or 100 mb minimal)
>
> 9) print suggestions
>
>
>
> 2017-08-28 15:10 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:
>
> > Igor, can you please describe the algorithm with all the thresholds?
> >
> > On Mon, Aug 28, 2017 at 4:56 AM, Seliverstov Igor <[hidden email]>
> > wrote:
> >
> > > The suggestion here is based on initial settings, and it's so because
> > there
> > > is no other nodes on the host in the example.
> > >
> > > The algorithm tries to preserve the original ratio of memory policies
> > > keeping numbers reasonable (for example after some thresshold it will
> > > suggest not to use several memory policies if there is not enough of
> > memory
> > > for all of them) and taking into consideration nodes count on the host,
> > > each jvm heap, needed memory for OS, etc
> > >
> > > 2017-08-28 14:38 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:
> > >
> > > > Looks good, but why in the example provided are we suggesting 8GB? 2
> > > nodes
> > > > with 8GB will completely exhaust the available memory. I would
> suggest
> > 6
> > > or
> > > > 7GB.
> > > >
> > > > Also, why 100MB for default policy. Anything under 1GB seems too
> small.
> > > >
> > > > Can you please comment?
> > > >
> > > > D.
> > > >
> > > > On Mon, Aug 28, 2017 at 3:31 AM, Seliverstov Igor <
> > [hidden email]>
> > > > wrote:
> > > >
> > > > > One more example of possible warning:
> > > > >
> > > > > -----------------------------------------------------
> > > > > Excessive memory usage by Ignite node process (performance may
> drop)
> > > > > [requested=44613MB, available=15942MB].
> > > > >
> > > > > Please tune the folowing settings as suggested:
> > > > >   MemoryPolicyConfiguration.initialSize for bigPlc: 8102MB
> > > > >   MemoryPolicyConfiguration.maxSize     for bigPlc: 8102MB
> > > > >   MemoryPolicyConfiguration.initialSize for dfltPlc: 100MB
> > > > >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 100MB
> > > > >
> > > > > Current settings:
> > > > >   Java Heap  maxSize: 3543MB
> > > > >   Java Heap initSize: 250MB
> > > > >   MemoryPolicyConfiguration.initialSize for bigPlc: 256MB
> > > > >   MemoryPolicyConfiguration.maxSize     for bigPlc: 40960MB
> > > > >   MemoryPolicyConfiguration.initialSize for dfltPlc: 10MB
> > > > >   MemoryPolicyConfiguration.maxSize     for dfltPlc: 10MB
> > > > >   The overall expected memory usage by all Ignite nodes on the
> host:
> > > > > 44613MB
> > > > > -----------------------------------------------------
> > > > >
> > > > > Your thoughts?
> > > > >
> > > > > 2017-08-28 5:06 GMT+03:00 Denis Magda <[hidden email]>:
> > > > >
> > > > > > Guys,
> > > > > >
> > > > > > ML lib profile is missing in 2.1 release! That must be fixed and
> > > rolled
> > > > > > out in this emergency release:
> > > > > > https://issues.apache.org/jira/browse/IGNITE-6193 <
> > > > > > https://issues.apache.org/jira/browse/IGNITE-6193>
> > > > > >
> > > > > > Oleg, Yuri, please step in and handle the issue.
> > > > > >
> > > > > > BTW, who is considered to be the release manager of this release?
> > > > > >
> > > > > > —
> > > > > > Denis
> > > > > >
> > > > > > > On Aug 25, 2017, at 2:29 PM, Dmitriy Setrakyan <
> > > > [hidden email]>
> > > > > > wrote:
> > > > > > >
> > > > > > > I like the format proposed by Denis, very clear.
> > > > > > >
> > > > > > > However, I also do not understand why a user should change the
> > size
> > > > of
> > > > > > some
> > > > > > > system cache. How would a user ever know what value to put
> there?
> > > > This
> > > > > > > value should be configured by Ignite automatically.
> > > > > > >
> > > > > > > D.
> > > > > > >
> > > > > > > On Fri, Aug 25, 2017 at 2:24 PM, Denis Magda <
> [hidden email]>
> > > > > wrote:
> > > > > > >
> > > > > > >> Igor,
> > > > > > >>
> > > > > > >> Let me suggest this format.
> > > > > > >>
> > > > > > >> ---------------------------------------------
> > > > > > >> Excessive memory usage by Ignite node process (performance may
> > > drop)
> > > > > > >> [requested=29251MB, available=15942MB]
> > > > > > >>
> > > > > > >> Please tune the following settings:
> > > > > > >>  [MemoryConfiguration.defaultMemoryPolicySize = suggested
> > value]
> > > > > > >>  MemoryConfiguration.systemCacheMaxSize = suggested value
> > > > > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_1} =
> > > suggested
> > > > > > >> value]
> > > > > > >>  [MemoryPolicyConfiguration.maxSize for {policy_name_2} =
> > > suggested
> > > > > > >> value]
> > > > > > >>
> > > > > > >> Current settings:
> > > > > > >>   [DefaultMemoryPolicySize = value]
> > > > > > >>   [{policy_name_1} size = value]
> > > > > > >>   [{policy_name_1} size = value]
> > > > > > >>   SystemCacheInitialSize = value
> > > > > > >>   SystemCacheMaxSize = value
> > > > > > >>   Java Heap Init Size = value
> > > > > > >>   Java Heap Max Size = value
> > > > > > >>
> > > > > > >> The overall memory usage by all Ignite nodes on the host:
> value
> > > > > > >> -------------------------------------------
> > > > > > >>
> > > > > > >> Records in […] are optional. If custom memory policy is not
> set
> > or
> > > > the
> > > > > > >> default memory policy is overridden the output will miss some
> of
> > > the
> > > > > > rows.
> > > > > > >>
> > > > > > >> As for systemCacheMaxSize, it should be show ONLY if the
> > parameter
> > > > was
> > > > > > set
> > > > > > >> explicitly by user code. Otherwise, the platform should be
> wise
> > > > enough
> > > > > > to
> > > > > > >> instantiate it properly depending on the host memory usage.
> > > > > > >>
> > > > > > >> —
> > > > > > >> Denis
> > > > > > >>
> > > > > > >>> On Aug 25, 2017, at 1:49 PM, Seliverstov Igor <
> > > > [hidden email]>
> > > > > > >> wrote:
> > > > > > >>>
> > > > > > >>> The message without logging layout:
> > > > > > >>>
> > > > > > >>> Not enough memory for current process [required=29251MB,
> > > > > > >> available=15942MB].
> > > > > > >>> Please change MemoryConfiguration.systemCacheMaxSize and
> > > > > > >>> MemoryConfiguration.defaultMemoryPolicySize to decrease
> memory
> > > > > > allocated
> > > > > > >>> for each node.
> > > > > > >>>
> > > > > > >>> Current settings:
> > > > > > >>>  HeapInit=250MB
> > > > > > >>>  HeapMax=3543MB
> > > > > > >>>  DefaultMemoryPolicySize=12753MB
> > > > > > >>>  SystemCacheInitialSize=40MB
> > > > > > >>>  SystemCacheMaxSize=100MB
> > > > > > >>>
> > > > > > >>> Other ignite instanses on the server require: 12853MB
> > > > > > >>>
> > > > > > >>> I think it's make sense to describe what these numbers are
> > > consist
> > > > > of.
> > > > > > >>>
> > > > > > >>> We simple say which parameters have an impact on how much
> > memory
> > > > the
> > > > > > >>> instance needs and their (parameters) actual values.
> > > > > > >>>
> > > > > > >>> Also we notice that more than one Ignite instance are ran on
> > the
> > > > > server
> > > > > > >> or
> > > > > > >>> workstation and it also consumes memory.
> > > > > > >>>
> > > > > > >>> 25 авг. 2017 г. 21:30 пользователь "Dmitriy Setrakyan" <
> > > > > > >>> [hidden email]> написал:
> > > > > > >>>
> > > > > > >>>> Igor, what is this flood of WARN messaging coming after the
> > > text?
> > > > > Are
> > > > > > we
> > > > > > >>>> really going to print this whole thing out?
> > > > > > >>>>
> > > > > > >>>> On Fri, Aug 25, 2017 at 9:49 AM, Seliverstov Igor <
> > > > > > [hidden email]
> > > > > > >>>
> > > > > > >>>> wrote:
> > > > > > >>>>
> > > > > > >>>>> This message appears on topology change in case the
> available
> > > > > memory
> > > > > > is
> > > > > > >>>>> exceeded
> > > > > > >>>>>
> > > > > > >>>>> 2017-08-25 19:47 GMT+03:00 Seliverstov Igor <
> > > > [hidden email]
> > > > > >:
> > > > > > >>>>>
> > > > > > >>>>>> An example of current impl:
> > > > > > >>>>>>
> > > > > > >>>>>>
> > > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Not enough
> > > memory
> > > > > for
> > > > > > >>>>>> current process [required=29251MB, available=15942MB].
> > > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Please
> change
> > > > > > >>>>>> MemoryConfiguration.systemCacheMaxSize and
> > > MemoryConfiguration.
> > > > > > >>>>> defaultMemoryPolicySize
> > > > > > >>>>>> to decrease memory allocated for each node.
> > > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Current
> > > settings:
> > > > > > >>>>>> [2017-08-25 19:44:37,740][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >  HeapInit=250MB
> > > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > >  HeapMax=3543MB
> > > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > > >>>> DefaultMemoryPolicySize=
> > > > > > >>>>>> 12753MB
> > > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > > >>>>>> SystemCacheInitialSize=40MB
> > > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > > >>>> SystemCacheMaxSize=100MB
> > > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager] Other
> ignite
> > > > > > instanses
> > > > > > >>>> on
> > > > > > >>>>>> the server require: 12853MB
> > > > > > >>>>>> [2017-08-25 19:44:37,741][WARN ][disco-event-worker-#29%
> > > > internal.
> > > > > > >>>>>> GridHomePathSelfTest0%][GridDiscoveryManager]
> > > > > > >>>>>>
> > > > > > >>>>>>
> > > > > > >>>>>> 2017-08-25 17:40 GMT+03:00 Sergey Kozlov <
> > > [hidden email]
> > > > >:
> > > > > > >>>>>>
> > > > > > >>>>>>> I suppose we should not forget JVM heap size and suggest
> to
> > > > > reduce
> > > > > > >>>> both
> > > > > > >>>>>>> options
> > > > > > >>>>>>>
> > > > > > >>>>>>> On Fri, Aug 25, 2017 at 5:24 PM, Dmitriy Setrakyan <
> > > > > > >>>>> [hidden email]
> > > > > > >>>>>>>>
> > > > > > >>>>>>> wrote:
> > > > > > >>>>>>>
> > > > > > >>>>>>>> Igor, I would change the message. How about this:
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> Required RAM size is larger than total physical memory
> > > > available
> > > > > > for
> > > > > > >>>>> OS.
> > > > > > >>>>>>>>> Please change MemoryConfiguration.WhichProperty and
> > > > > > >>>>>>>>> MemoryPolicyConfiguration.WhichProperty to decrease
> > memory
> > > > > > >>>>> allocated
> > > > > > >>>>>>> for
> > > > > > >>>>>>>>> each node.
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> Also, can we calculate what the memory size allocated
> for
> > > each
> > > > > > node
> > > > > > >>>>>>> should
> > > > > > >>>>>>>> be? In that case we should suggest it.
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> D.
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> On Fri, Aug 25, 2017 at 7:20 AM, Seliverstov Igor <
> > > > > > >>>>> [hidden email]
> > > > > > >>>>>>>>
> > > > > > >>>>>>>> wrote:
> > > > > > >>>>>>>>
> > > > > > >>>>>>>>> What do you, guys think about next warning?
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> [2017-08-25 17:17:04,718][INFO
> > > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > > GridHomePat
> > > > > > >>>>>>> hSelfTest0]
> > > > > > >>>>>>>>> System cache's MemoryPolicy size is configured to 40
> MB.
> > > Use
> > > > > > >>>>>>>>> MemoryConfiguration.systemCacheMemorySize property to
> > > change
> > > > > the
> > > > > > >>>>>>>> setting.
> > > > > > >>>>>>>>> [2017-08-25 17:17:04,718][WARN
> > > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > > GridHomePat
> > > > > > >>>>>>> hSelfTest0]
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>>>>> Required RAM size is larger than total physical
> memory
> > > > > > >>>> available
> > > > > > >>>>>>> for
> > > > > > >>>>>>>>> OS.
> > > > > > >>>>>>>>>>>> Check your configuration to avoid swap partition
> > usage.
> > > > > > >>>>>>>>>>>> Use MemoryConfiguration and
> MemoryPolicyConfiguration
> > to
> > > > > > >>>> change
> > > > > > >>>>>>> the
> > > > > > >>>>>>>>> settings.
> > > > > > >>>>>>>>>>>> Physical memory [required=16397MB,
> available=15942MB]
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> [2017-08-25 17:17:04,726][WARN
> > > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > > GridHomePat
> > > > > > >>>>>>> hSelfTest0]
> > > > > > >>>>>>>>> Peer class loading is enabled (disable it in production
> > for
> > > > > > >>>>>>> performance
> > > > > > >>>>>>>> and
> > > > > > >>>>>>>>> deployment consistency reasons)
> > > > > > >>>>>>>>> [2017-08-25 17:17:04,726][INFO
> > > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > > GridHomePat
> > > > > > >>>>>>> hSelfTest0]
> > > > > > >>>>>>>>> Configured caches [in 'sysMemPlc' memoryPolicy:
> > > > > > >>>>> ['ignite-sys-cache']]
> > > > > > >>>>>>>>> [2017-08-25 17:17:04,731][INFO
> > > > > > >>>>>>>>> ][test-runner-#1%internal.GridHomePathSelfTest%][
> > > GridHomePat
> > > > > > >>>>>>> hSelfTest0]
> > > > > > >>>>>>>>> 3-rd party licenses can be found at:
> > > > > > >>>>>>>>> /home/gvvinblade/projects/
> ignite/incubator-ignite/libs/
> > > > > licenses
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>> 2017-08-25 13:26 GMT+03:00 Yakov Zhdanov <
> > > > [hidden email]
> > > > > >:
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>>> Agree, let's release new version including tickets
> > > mentioned
> > > > > by
> > > > > > >>>>>>> Denis
> > > > > > >>>>>>>> and
> > > > > > >>>>>>>>>> Ivan.
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>> --Yakov
> > > > > > >>>>>>>>>>
> > > > > > >>>>>>>>>
> > > > > > >>>>>>>>
> > > > > > >>>>>>>
> > > > > > >>>>>>>
> > > > > > >>>>>>>
> > > > > > >>>>>>> --
> > > > > > >>>>>>> Sergey Kozlov
> > > > > > >>>>>>> GridGain Systems
> > > > > > >>>>>>> www.gridgain.com
> > > > > > >>>>>>>
> > > > > > >>>>>>
> > > > > > >>>>>>
> > > > > > >>>>>
> > > > > > >>>>
> > > > > > >>
> > > > > > >>
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>
123