[prev in list] [next in list] [prev in thread] [next in thread] 

List:       syslog-ng
Subject:    Re: [syslog-ng] =?utf-8?q?Finding_feature_owner_for_Kafka_source_proj?=
From:       Laszlo Budai <laszlo.budai () outlook ! com>
Date:       2016-05-22 4:06:40
Message-ID: BAY404-EAS68EF087A66C10B6938E7A08E4D0 () phx ! gbl
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

[Attachment #4 (text/plain)]


Sent from my HTC

----- Reply message -----
From: "Vincent Bernat" <bernat@luffy.cx>
To: "Laszlo Budai" <laszlo.budai@outlook.com>
Cc: "Syslog-ng users' and developers' mailing list" <syslog-ng@lists.balabit.hu>, \
                "Vithulan MV" <vithulanmv.12@cse.mrt.ac.lk>
Subject: [syslog-ng] Finding feature owner for Kafka source project
Date: Sat, May 21, 2016 23:58

❦ 21 mai 2016 11:58 -0700, Laszlo Budai <laszlo.budai@outlook.com> :

> As you mentioned we have Kafka destination driver written in C in
> syslog-ng incubator, so users can use it. It should not be a problem
> if a feature is available only in the syslog-ng incubator.

Well, it is because most users will never use it.
I don't know why... maybe we should make it clear what is incubator.For me it \
contains two kind of features:1) experimental features that are not really tested, \
and maybe the code would also neee some polishment before merge into syslog-ng \
master2) well implemented features that are  using some unstable dependencies> As the \
Kafka C client is stable enough, I don't see any problems in
> integrating it to the syslog-ng master (just someone should create a
> pull request).

Good to know. That's not what you told me in August. I will fix the
performance limitations and do a PR for master.I said that my plan is to keep the C \
kafka dst in incubator. Last year ;)But Kafka C lib seems to be a stable lib during \
the last 8-9 months which makes it possible to move forward from incubator.  L.-- 
Make it right before you make it faster.
- The Elements of Programming Style (Kernighan & Plauger)


[Attachment #5 (text/html)]


<div style="font-family: 'Calibri', 'sans-serif';"><div dir="ltr">
<div dir="ltr"><br></div><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr">Sent \
from my HTC</div></div>

</div><br><div id="htc_header">----- Reply message -----<br>From: &quot;Vincent \
Bernat&quot; &lt;bernat@luffy.cx&gt;<br>To: &quot;Laszlo Budai&quot; \
&lt;laszlo.budai@outlook.com&gt;<br>Cc: &quot;Syslog-ng users&#39; and \
developers&#39; mailing list&quot; &lt;syslog-ng@lists.balabit.hu&gt;, &quot;Vithulan \
MV&quot; &lt;vithulanmv.12@cse.mrt.ac.lk&gt;<br>Subject: [syslog-ng] Finding feature \
owner for Kafka source project<br>Date: Sat, May 21, 2016 23:58</div></div><br><div \
dir="ltr"><pre style="word-wrap: break-word; white-space: pre-wrap;"> ❦ 21 mai 2016 \
11:58 -0700, Laszlo Budai &lt;laszlo.budai@outlook.com&gt;&nbsp;:

&gt; As you mentioned we have Kafka destination driver written in C in
&gt; syslog-ng incubator, so users can use it. It should not be a problem
&gt; if a feature is available only in the syslog-ng incubator.

Well, it is because most users will never use it.</pre><pre style="word-wrap: \
break-word; white-space: pre-wrap;"><br></pre><pre style="word-wrap: break-word; \
white-space: pre-wrap;">I don't know why... maybe we should make it clear what is \
incubator.</pre><pre style="word-wrap: break-word; white-space: pre-wrap;">For me it \
contains two kind of features:</pre><pre style="word-wrap: break-word; white-space: \
pre-wrap;">1) experimental features that are not really tested, and maybe the code \
would also neee some polishment before merge into syslog-ng master</pre><pre \
style="word-wrap: break-word; white-space: pre-wrap;">2) well implemented features \
that are  using some unstable dependencies</pre><pre style="word-wrap: break-word; \
white-space: pre-wrap;">&gt; As the Kafka C client is stable enough, I don't see any \
problems in &gt; integrating it to the syslog-ng master (just someone should create a
&gt; pull request).

Good to know. That's not what you told me in August. I will fix the
performance limitations and do a PR for master.</pre><pre style="word-wrap: \
break-word; white-space: pre-wrap;">I said that my plan is to keep the C kafka dst in \
incubator. Last year ;)</pre><pre style="word-wrap: break-word; white-space: \
pre-wrap;">But Kafka C lib seems to be a stable lib during the last 8-9 months which \
makes it possible to move forward from incubator. </pre><pre style="word-wrap: \
break-word; white-space: pre-wrap;"><br></pre><pre style="word-wrap: break-word; \
white-space: pre-wrap;">L.</pre><pre style="word-wrap: break-word; white-space: \
pre-wrap;">--  Make it right before you make it faster.
            - The Elements of Programming Style (Kernighan &amp; Plauger)
</pre></div>



______________________________________________________________________________
Member info: https://lists.balabit.hu/mailman/listinfo/syslog-ng
Documentation: http://www.balabit.com/support/documentation/?product=syslog-ng
FAQ: http://www.balabit.com/wiki/syslog-ng-faq



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic