[
https://issues.apache.org/jira/browse/DL-172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15802368#comment-15802368
]
ASF GitHub Bot commented on DL-172:
-----------------------------------
Github user sijie commented on the issue:
https://github.com/apache/incubator-distributedlog/pull/99
@gerritsundaram looks like a findbugs warming. you might consider excluding
the generated code.
regarding whether using thrift or protobuf, can you raise a discussion
email thread for that?
> Provide a IDL definition for stream metadata and segment metadata
> -----------------------------------------------------------------
>
> Key: DL-172
> URL: https://issues.apache.org/jira/browse/DL-172
> Project: DistributedLog
> Issue Type: New Feature
> Components: distributedlog-protocol
> Reporter: Gerrit Sundaram
> Assignee: Gerrit Sundaram
> Fix For: 0.5.0
>
>
> Currently there is not stream metadata structure and log segment metadata is
> a customized string structure. it would be good to use any IDL like thrift,
> protobuf to store the metadata like what bookkeeper is doing.
> The reason to choose protobuf is
> - because bookkeeper is using protobuf, it is okay to use same stuff.
> - the thrift version that DL is using is a bit too old and hard to bump
> because it seems the version is customized for finagle. using protobuf is
> easier to support other language clients.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
