You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Sergey Kozlov <sk...@gridgain.com> on 2015/11/19 09:58:11 UTC

Ignite camel streamer

Hi Raul


I just found a few minor issues for your Ignite Camel streamer module if I
build the fabric edition:

1. The module has no README.txt (/libs/optional/ignite-camel/README.txt).
2. The module has no Apache License
(/libs/optional/ignite-camel/licenses/apache-2.0.txt).

Could you take a look and  fix it?

Thanks!

-- 
Sergey Kozlov

Re: Ignite camel streamer

Posted by Dmitriy Setrakyan <ds...@apache.org>.
On Thu, Nov 19, 2015 at 3:51 AM, Raul Kripalani <ra...@apache.org> wrote:

> Hi,
>
> I added the README.txt and the license.txt in commit
> 480d457a724800c1afa4aadf3fc9ceda7f9a217f.
>
> For the record, you must not call something an "issue" and request "a fix"
> when there was no requirement in the Wiki to begin with (it was added
> earlier today by Anton Vinogradov).
>

Raul, seems like Anton saw that the readme was missing in this module,
while all other modules have it, updated the Wiki, and asked you to make
the changes. I am personally grateful to Anton for testing out the
functionality and updating the Wiki.

I can see that you might have a problem with a choice of words, but you
must understand that you are dealing with a community where English often
is not a second, but a third language. Some folks are using Google
Translate to help them communicate. Not everything translates correctly
from different languages, and every now and then a bad translation will
slip in.

I am a bit concerned that such commentary may discourage the community from
communicating in English altogether due to a constant fear of saying
something wrong, which will consequently slow down the contribution
process. I would like to encourage all the community members not to focus
on the exact choice of words, but rather on the essence and the end result.
The end result here, in my view, is that the project becomes better, which
is most important.

Thanks,
D.


>
> Let's use correct terminology hereafter...
>
> Regards,
>
> *Raúl Kripalani*
> PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and
> Messaging Engineer
> http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani
> http://blog.raulkr.net | twitter: @raulvk
>
> On Thu, Nov 19, 2015 at 8:58 AM, Sergey Kozlov <sk...@gridgain.com>
> wrote:
>
> > Hi Raul
> >
> >
> > I just found a few minor issues for your Ignite Camel streamer module if
> I
> > build the fabric edition:
> >
> > 1. The module has no README.txt (/libs/optional/ignite-camel/README.txt).
> > 2. The module has no Apache License
> > (/libs/optional/ignite-camel/licenses/apache-2.0.txt).
> >
> > Could you take a look and  fix it?
> >
> > Thanks!
> >
> > --
> > Sergey Kozlov
> >
>

Re: Ignite camel streamer

Posted by Raul Kripalani <ra...@apache.org>.
Hi,

I added the README.txt and the license.txt in commit
480d457a724800c1afa4aadf3fc9ceda7f9a217f.

For the record, you must not call something an "issue" and request "a fix"
when there was no requirement in the Wiki to begin with (it was added
earlier today by Anton Vinogradov).

Let's use correct terminology hereafter...

Regards,

*Raúl Kripalani*
PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and
Messaging Engineer
http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani
http://blog.raulkr.net | twitter: @raulvk

On Thu, Nov 19, 2015 at 8:58 AM, Sergey Kozlov <sk...@gridgain.com> wrote:

> Hi Raul
>
>
> I just found a few minor issues for your Ignite Camel streamer module if I
> build the fabric edition:
>
> 1. The module has no README.txt (/libs/optional/ignite-camel/README.txt).
> 2. The module has no Apache License
> (/libs/optional/ignite-camel/licenses/apache-2.0.txt).
>
> Could you take a look and  fix it?
>
> Thanks!
>
> --
> Sergey Kozlov
>