You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/05/25 09:01:04 UTC

[jira] [Commented] (FLINK-6714) Operator state backend should set user classloader as context classloader when snapshotting

    [ https://issues.apache.org/jira/browse/FLINK-6714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16024423#comment-16024423 ] 

ASF GitHub Bot commented on FLINK-6714:
---------------------------------------

GitHub user tzulitai opened a pull request:

    https://github.com/apache/flink/pull/3987

    [FLINK-6714] [runtime] Use user classloader for operator state copying on snapshots

    Now that the operator state backend creates a deep copy of the state during the synchronous part of async checkpoints, it needs to set the user classloader as the thread context classloader, otherwise serializers that uses serialization for copying will use the wrong classloader when deserializing the copy.
    
    A test is added to {{OperatorStateBackendTest}} to verify that the user classloader is properly used on copy.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tzulitai/flink FLINK-6714

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/3987.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3987
    
----
commit 516fd085d640d56ea2ef7be7f0b11f6ad43e92f5
Author: Tzu-Li (Gordon) Tai <tz...@apache.org>
Date:   2017-05-25T08:54:08Z

    [FLINK-6714] [runtime] Use user classloader for operator state copying on snapshots

----


> Operator state backend should set user classloader as context classloader when snapshotting
> -------------------------------------------------------------------------------------------
>
>                 Key: FLINK-6714
>                 URL: https://issues.apache.org/jira/browse/FLINK-6714
>             Project: Flink
>          Issue Type: Bug
>          Components: State Backends, Checkpointing
>            Reporter: Tzu-Li (Gordon) Tai
>            Assignee: Tzu-Li (Gordon) Tai
>            Priority: Blocker
>
> Now that the operator state backend creates a deep copy of the state during the synchronous part of async checkpoints, it needs to set the user classloader as the thread context classloader, otherwise serializers that uses serialization for copying will use the wrong classloader when deserializing the copy.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)