You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Jay Kreps (JIRA)" <ji...@apache.org> on 2014/07/22 00:48:38 UTC
[jira] [Resolved] (KAFKA-322) Remove one-off Send objects
[ https://issues.apache.org/jira/browse/KAFKA-322?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jay Kreps resolved KAFKA-322.
-----------------------------
Resolution: Fixed
This was fixed a long while back, cleaning up.
> Remove one-off Send objects
> ---------------------------
>
> Key: KAFKA-322
> URL: https://issues.apache.org/jira/browse/KAFKA-322
> Project: Kafka
> Issue Type: Improvement
> Components: core
> Affects Versions: 0.8.0
> Reporter: Jay Kreps
> Assignee: Jay Kreps
> Priority: Minor
> Labels: replication
> Fix For: 0.9.0
>
>
> We seem to be accumulating a bunch of unnecessary classes that implement Send. I am not sure why people are doing this. Example:
> ProducerResponseSend.scala
> It is not at all clear why we would add a custom send object for each request/response type. They all do the same thing. The only reason for having the concept of a Send object was to allow two implementations: ByteBufferSend and MessageSetSend, the later let's us abstract over the difference between a normal write and a sendfile() call.
> I think we can refactory ByteBufferSend to take one or more ByteBuffers instead of just one and delete all of these one-offs.
--
This message was sent by Atlassian JIRA
(v6.2#6252)