[ 
https://issues.apache.org/jira/browse/CASSSIDECAR-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17904298#comment-17904298
 ] 

Jon Haddad commented on CASSSIDECAR-158:
----------------------------------------

I think it's great that the CDC reader is coming along, but this was merged 
without a single performance test and no feedback other than teammates who both 
+1'ed the patch without any outside input.

I realize we're not in the habit of reverting changes, but to merge such a big 
piece of functionality in without doing *any* public performance testing is not 
OK.  

I'm taking this to the ML, because at the moment I'm inclined to -1 the patch 
and ask for it to be reverted. 

> Adding APIs required for CDC
> ----------------------------
>
>                 Key: CASSSIDECAR-158
>                 URL: https://issues.apache.org/jira/browse/CASSSIDECAR-158
>             Project: Sidecar for Apache Cassandra
>          Issue Type: New Feature
>          Components: Rest API
>            Reporter: Jyothsna Konisa
>            Assignee: Jyothsna Konisa
>            Priority: Urgent
>             Fix For: 1.0
>
>          Time Spent: 4h 50m
>  Remaining Estimate: 0h
>
> Adding the following 2 APIs cassandra sidecar which are required for adding 
> Change Data Capture feature into C*.
>  - Listing CDC commitlogs/segments
>  - Streaming Commitlogs/segments
> CEP to CDC : 
> [https://cwiki.apache.org/confluence/display/CASSANDRA/CEP-44%3A+Kafka+integration+for+Cassandra+CDC+using+Sidecar]
> Discuss thread : 
> [https://lists.apache.org/thread/8k6njsnvdbmjb6jhyy07o1s7jz8xp1qg]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to