[jira] [Commented] (FLINK-24854) StateHandleSerializationTest unit test error

2021-11-23 Thread Stephan Ewen (Jira)


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

Stephan Ewen commented on FLINK-24854:
--

Thanks for reporting this.
It looks like [~chesnay] already added a fix for this. Could you double check 
that this is the same issue, [~chesnay]?

> StateHandleSerializationTest unit test error
> 
>
> Key: FLINK-24854
> URL: https://issues.apache.org/jira/browse/FLINK-24854
> Project: Flink
>  Issue Type: Bug
>  Components: Tests
>Affects Versions: 1.14.0
>Reporter: zlzhang0122
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.15.0
>
>
> StateHandleSerializationTest.ensureStateHandlesHaveSerialVersionUID() will 
> fail beacuse RocksDBStateDownloaderTest has an anonymous class of 
> StreamStateHandle, and this class is a subtype of StateObject, since the 
> class is an anonymous, the assertFalse will fail as well as this unit test.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (FLINK-24854) StateHandleSerializationTest unit test error

2021-11-22 Thread zlzhang0122 (Jira)


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

zlzhang0122 commented on FLINK-24854:
-

[~sewen] Hi, what's your opinion? Any suggestion is very appreciate!

> StateHandleSerializationTest unit test error
> 
>
> Key: FLINK-24854
> URL: https://issues.apache.org/jira/browse/FLINK-24854
> Project: Flink
>  Issue Type: Bug
>  Components: Tests
>Affects Versions: 1.14.0
>Reporter: zlzhang0122
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.15.0
>
>
> StateHandleSerializationTest.ensureStateHandlesHaveSerialVersionUID() will 
> fail beacuse RocksDBStateDownloaderTest has an anonymous class of 
> StreamStateHandle, and this class is a subtype of StateObject, since the 
> class is an anonymous, the assertFalse will fail as well as this unit test.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Commented] (FLINK-24854) StateHandleSerializationTest unit test error

2021-11-10 Thread zlzhang0122 (Jira)


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

zlzhang0122 commented on FLINK-24854:
-

I think we can just remove the assertFalse sentence and add serialVersionUID 
fot the anonymous class of StreamStateHandle in RocksDBStateDownloaderTest, 
with this change, everything will be fine.

> StateHandleSerializationTest unit test error
> 
>
> Key: FLINK-24854
> URL: https://issues.apache.org/jira/browse/FLINK-24854
> Project: Flink
>  Issue Type: Bug
>  Components: Tests
>Affects Versions: 1.14.0
>Reporter: zlzhang0122
>Priority: Major
> Fix For: 1.15.0
>
>
> StateHandleSerializationTest.ensureStateHandlesHaveSerialVersionUID() will 
> fail beacuse RocksDBStateDownloaderTest has an anonymous class of 
> StreamStateHandle, and this class is a subtype of StateObject, since the 
> class is an anonymous, the assertFalse will fail as well as this unit test.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)