As I mentioned in the KIP, this transformer is slightly different from the 
current MaskField SMT. 

> Currently there exists a MaskField SMT but that would completely remove the 
> value by setting it to an equivalent null value. One problem with this would 
> be that you’d not be able to know in the case of say a password going through 
> the mask transform it would become "" which could mean that no password was 
> present in the message, or it was removed. However this hash transformer 
> would remove this ambiguity if that makes sense. The proposed hash functions 
> would be MD5, SHA1, SHA256. which are all supported via MessageDigest.

Given this take on things do you still think there would be value in this smt?


Brandon Brown 
> On Oct 15, 2020, at 12:36 PM, Ning Zhang <ning2008w...@gmail.com> wrote:
> 
> Hello, I think this SMT feature is parallel to 
> https://docs.confluent.io/current/connect/transforms/index.html
> 
>> On 2020/10/15 15:24:51, Brandon Brown <bran...@bbrownsound.com> wrote: 
>> Bumping this thread.
>> Please take a look at the KIP and vote or let me know if you have any 
>> feedback.
>> 
>> KIP: 
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-665%3A+Kafka+Connect+Hash+SMT
>> 
>> Proposed: https://github.com/apache/kafka/pull/9057
>> 
>> Thanks
>> 
>> Brandon Brown
>> 
>>>> On Oct 8, 2020, at 10:30 PM, Brandon Brown <bran...@bbrownsound.com> wrote:
>>> 
>>> Just wanted to give another bump on this and see if anyone had any 
>>> comments. 
>>> 
>>> Thanks!
>>> 
>>> Brandon Brown
>>> 
>>>> On Oct 1, 2020, at 9:11 AM, "bran...@bbrownsound.com" 
>>>> <bran...@bbrownsound.com> wrote:
>>>> 
>>>> Hey Kafka Developers,
>>>> 
>>>> I’ve created the following KIP and updated it based on feedback from 
>>>> Mickael. I was wondering if we could get a vote on my proposal and move 
>>>> forward with the proposed pr.
>>>> 
>>>> Thanks so much!
>>>> -Brandon
>> 

Reply via email to