[
https://issues.apache.org/jira/browse/BEAM-3279?focusedWorklogId=214096&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-214096
]
ASF GitHub Bot logged work on BEAM-3279:
----------------------------------------
Author: ASF GitHub Bot
Created on: 15/Mar/19 23:01
Start Date: 15/Mar/19 23:01
Worklog Time Spent: 10m
Work Description: kennknowles commented on pull request #8071:
[BEAM-3279] Deprecate and remove Coder.consistentWithEquals
URL: https://github.com/apache/beam/pull/8071#discussion_r266170461
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/coders/BigEndianShortCoder.java
##########
@@ -62,16 +62,6 @@ public Short decode(InputStream inStream) throws
IOException, CoderException {
@Override
public void verifyDeterministic() {}
- /**
- * {@inheritDoc}
- *
- * @return {@code true}. This coder is injective.
- */
- @Override
- public boolean consistentWithEquals() {
Review comment:
Does each of these need an implementation of structuralValue? Are there
tests that fail if it is set to the wrong thing?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 214096)
Time Spent: 20m (was: 10m)
> Deprecate and remove Coder consistentWithEquals in favor of overriding
> structuredValue
> --------------------------------------------------------------------------------------
>
> Key: BEAM-3279
> URL: https://issues.apache.org/jira/browse/BEAM-3279
> Project: Beam
> Issue Type: Improvement
> Components: sdk-java-core
> Reporter: Kenneth Knowles
> Assignee: Alexander Savchenko
> Priority: Minor
> Labels: starter
> Time Spent: 20m
> Remaining Estimate: 0h
>
> Summary of discussion linked below:
> consistentWithEquals() is redundant w.r.t. structuralValue(), and should be
> deprecated. I think our mutation detectors are already using
> structuralValue(), so the work here would be to simply mark the method
> deprecated, remove all remaining overrides in the SDK, and document that
> overriding the method is a no-op.
> https://lists.apache.org/thread.html/8b2dcf09ba8e46b3c008293d99e4028d10463148b68326687dc29a4d@%3Cdev.beam.apache.org%3E
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
