[
https://issues.apache.org/jira/browse/CASSANDRA-754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12833816#action_12833816
]
Gary Dusbabek commented on CASSANDRA-754:
-----------------------------------------
I see your points, but avro at this point is little more than a proof of
concept, a long way from being usable for testing, let alone production.
Putting this code in will send the wrong message IMO.
If we allow multiple APIs per configuration, then lets make the listen-address
required and push that down into <API>. To discourage messy config files,
let's require all <API> specifications within a single parent element (think of
a good name), the same way we do with keyspaces.
That ValidatingParser should know API class bothers me. Why not push that
logic down into the setup methods, or make ValidatingParser generic enough to
do away with the API class name parameters? I realize there will probably be
some code duplication, but we parse integers and load InetAddresses all over
the place anyway.
> Pluggable API with a simple HTTP+JSON daemon
> --------------------------------------------
>
> Key: CASSANDRA-754
> URL: https://issues.apache.org/jira/browse/CASSANDRA-754
> Project: Cassandra
> Issue Type: New Feature
> Components: Core
> Reporter: Ted Zlatanov
> Priority: Minor
> Attachments: 001-CASSANDRA-754-api-reorg.patch,
> 001-CASSANDRA-754-api-reorg.patch, 002-CASSANDRA-754-thriftdaemon.patch,
> 002-CASSANDRA-754-thriftdaemon.patch, 003-CASSANDRA-754-configapi.patch,
> 003-CASSANDRA-754-configapi.patch, 004-CASSANDRA-754-onedaemon.patch,
> 005-CASSANDRA-754-avroconfig.patch, 006-CASSANDRA-754-daemonproperties.patch,
> 006-CASSANDRA-754-daemonproperties.patch,
> 007-CASSANDRA-754-stringproperties.patch,
> 100-CASSANDRA-754-rebased-api-reorg.patch, 101-CASSANDRA-754-binclass.patch
>
>
> Proposal of a simple pluggable API interface with a HTTP daemon implementation
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
