Hi Michal, Thanks a lot for your feedback.
Your statement about Heartbeat is fair and makes sense. I'll update the KIP accordingly. --Vahid From: Michal Borowiecki <michal.borowie...@openbet.com> To: users@kafka.apache.org, Vahid S Hashemian <vahidhashem...@us.ibm.com>, d...@kafka.apache.org Date: 06/13/2017 01:35 AM Subject: Re: [DISCUSS] KIP-163: Lower the Minimum Required ACL Permission of OffsetFetch Hi Vahid, +1 wrt OffsetFetch. The "Additional Food for Thought" mentions Heartbeat as a non-mutating action. I don't think that's true as the GroupCoordinator updates the latestHeartbeat field for the member and adds a new object to the heartbeatPurgatory, see completeAndScheduleNextHeartbeatExpiration() called from handleHeartbeat() NB added dev mailing list back into CC as it seems to have been lost along the way. Cheers, Michał On 12/06/17 18:47, Vahid S Hashemian wrote: Hi Colin, Thanks for the feedback. To be honest, I'm not sure either why Read was selected instead of Write for mutating APIs in the initial design (I asked Ewen on the corresponding JIRA and he seemed unsure too). Perhaps someone who was involved in the design can clarify. Thanks. --Vahid From: Colin McCabe <cmcc...@apache.org> To: users@kafka.apache.org Date: 06/12/2017 10:11 AM Subject: Re: [DISCUSS] KIP-163: Lower the Minimum Required ACL Permission of OffsetFetch Hi Vahid, I think you make a valid point that the ACLs controlling group operations are not very intuitive. This is probably a dumb question, but why are we using Read for mutating APIs? Shouldn't that be Write? The distinction between Describe and Read makes a lot of sense for Topics. A group isn't really something that you "read" from in the same way as a topic, so it always felt kind of weird there. best, Colin On Thu, Jun 8, 2017, at 11:29, Vahid S Hashemian wrote: Hi all, I'm resending my earlier note hoping it would spark some conversation this time around :) Thanks. --Vahid From: "Vahid S Hashemian" <vahidhashem...@us.ibm.com> To: dev <d...@kafka.apache.org>, "Kafka User" <users@kafka.apache.org> Date: 05/30/2017 08:33 AM Subject: KIP-163: Lower the Minimum Required ACL Permission of OffsetFetch Hi, I started a new KIP to improve the minimum required ACL permissions of some of the APIs: https://cwiki.apache.org/confluence/display/KAFKA/KIP-163%3A+Lower+the+Minimum+Required+ACL+Permission+of+OffsetFetch The KIP is to address KAFKA-4585. Feedback and suggestions are welcome! Thanks. --Vahid -- Michal Borowiecki Senior Software Engineer L4 T: +44 208 742 1600 +44 203 249 8448 E: michal.borowie...@openbet.com W: www.openbet.com OpenBet Ltd Chiswick Park Building 9 566 Chiswick High Rd London W4 5XT UK This message is confidential and intended only for the addressee. If you have received this message in error, please immediately notify the postmas...@openbet.com and delete it from your system as well as any copies. The content of e-mails as well as traffic data may be monitored by OpenBet for employment and security purposes. To protect the environment please do not print this e-mail unless necessary. OpenBet Ltd. Registered Office: Chiswick Park Building 9, 566 Chiswick High Road, London, W4 5XT, United Kingdom. A company registered in England and Wales. Registered no. 3134634. VAT no. GB927523612
