[GitHub] [samza] srinipunuru commented on issue #979: SAMZA-2110 : Support for deleting records from a table

[GitBox <gi...@apache.org>]

srinipunuru commented on issue #979: SAMZA-2110 : Support for deleting records from a table
URL: https://github.com/apache/samza/pull/979#issuecomment-477700449
 
 
   @atoomula 
   
   > * delete events from cc streams. eg: "select * from brooklin.stream", "select f1, f2 from brooklin.stream"
   
   With this new model there needs to be separate sql statement for delete. I think this model may be better longer term, because When we have delete from support. users need to write two sql statements on for propagating inserts and updates and other one for deletes. This model is inline with that.
   
   > * Non-explicit delete events. eg: "select f1, f2 from kafka.topic". In this case both f1 and f2 could be null. Deleting such an event from destination store is unclear.
   
   When user does select f1, f2 and when both f1 and f2 are null, it is possible that user wants us to insert a record with each fields being null.  Assuming that the behavior in this case should be delete is non-intuitive and also problematic because we may never be able to support the above requested behavior. 
    
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services