[
https://issues.apache.org/jira/browse/FLINK-7736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16196296#comment-16196296
]
ASF GitHub Bot commented on FLINK-7736:
---------------------------------------
Github user haohui commented on a diff in the pull request:
https://github.com/apache/flink/pull/4784#discussion_r143363797
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/webmonitor/history/ArchivedJson.java
---
@@ -55,6 +57,13 @@ public boolean equals(Object obj) {
}
}
+ public int hashCode() {
--- End diff --
Just use Objects.hash here. And it needs the override annotation.
> Fix some of the alerts raised by lgtm.com
> -----------------------------------------
>
> Key: FLINK-7736
> URL: https://issues.apache.org/jira/browse/FLINK-7736
> Project: Flink
> Issue Type: Improvement
> Reporter: Malcolm Taylor
> Assignee: Malcolm Taylor
>
> lgtm.com has identified a number of issues giving scope for improvement in
> the code: [https://lgtm.com/projects/g/apache/flink/alerts/?mode=list]
> This issue is to address some of the simpler ones. Some of these are quite
> clear bugs such as off-by-one errors. Others are areas where the code might
> be made clearer, such as use of a variable name which shadows another
> variable.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
