[
https://issues.apache.org/jira/browse/FLINK-888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14046091#comment-14046091
]
ASF GitHub Bot commented on FLINK-888:
--------------------------------------
Github user zentol commented on the pull request:
https://github.com/apache/incubator-flink/pull/44#issuecomment-47366280
if you use ```Assert.assertEquals(fullTuple2.f0, swappedTuple2.f1)``` the
test will fail if they are not the same, just as your current code. you save
six lines, and it's more obvious what you want to check.
your test is supposed to make sure that the swapped values are equal, thus
you should check exactly that.
> Add swap() method to Tuple2
> ---------------------------
>
> Key: FLINK-888
> URL: https://issues.apache.org/jira/browse/FLINK-888
> Project: Flink
> Issue Type: Improvement
> Reporter: GitHub Import
> Assignee: Tobias
> Labels: github-import
> Fix For: pre-apache
>
>
> In line with Scala
> ---------------- Imported from GitHub ----------------
> Url: https://github.com/stratosphere/stratosphere/issues/888
> Created by: [StephanEwen|https://github.com/StephanEwen]
> Labels: enhancement, java api,
> Milestone: Release 0.6 (unplanned)
> Created at: Thu May 29 14:49:49 CEST 2014
> State: open
--
This message was sent by Atlassian JIRA
(v6.2#6252)
