|
|
12
|
|
+1 from wildfly-camel
— thomas
|
|
|
+1
---
Luca Burgazzoli
On Mon, Jun 27, 2016 at 1:20 PM, Andrea Cosentino
< [hidden email]> wrote:
|
|
|
Anyway you can always use -1 value for
requestRequiredAcks parameter.
The values in the enum are just for docs purpose.
I'll update the enum on URIParam, but you're able to use -1.
--
Andrea Cosentino
----------------------------------
Apache Camel PMC Member
Apache Karaf Committer
Apache Servicemix Committer
Email: [hidden email]
Twitter: @oscerd2
Github: oscerd
On Monday, June 27, 2016 1:48 PM, Andrea Cosentino < [hidden email]> wrote:
That's right.
http://kafka.apache.org/documentation.html#producerconfigs-1 is a valid value for acks.
I'll commit a fix for this and maybe we'll need to cut the release again.
My mistake. Sorry.
--
Andrea Cosentino
----------------------------------
Apache Camel PMC Member
Apache Karaf Committer
Apache Servicemix Committer
Email: [hidden email]
Twitter: @oscerd2
Github: oscerd
On Monday, June 27, 2016 1:41 PM, Gerard Klijs < [hidden email]> wrote:
-1 since https://issues.apache.org/jira/browse/CAMEL-9978 will make it
impossible to use "-1" as value for the acks, which is what we currently
use.
On Mon, Jun 27, 2016 at 1:21 PM Luca Burgazzoli < [hidden email]>
wrote:
|
|
|
Hi
No that is not a blocker. You can type in any string value you want, also -1.
Its only the @UriParam that has a enum that suggests it must be
"0,1,all". But there is no enforced validation on that.
On Mon, Jun 27, 2016 at 1:48 PM, Andrea Cosentino
< [hidden email]> wrote:
> That's right.
>
> http://kafka.apache.org/documentation.html#producerconfigs>
> -1 is a valid value for acks.
>
> I'll commit a fix for this and maybe we'll need to cut the release again.
>
> My mistake. Sorry.
> --
> Andrea Cosentino
> ----------------------------------
> Apache Camel PMC Member
> Apache Karaf Committer
> Apache Servicemix Committer
> Email: [hidden email]
> Twitter: @oscerd2
> Github: oscerd
>
>
>
> On Monday, June 27, 2016 1:41 PM, Gerard Klijs < [hidden email]> wrote:
> -1 since https://issues.apache.org/jira/browse/CAMEL-9978 will make it
> impossible to use "-1" as value for the acks, which is what we currently
> use.
>
>
> On Mon, Jun 27, 2016 at 1:21 PM Luca Burgazzoli < [hidden email]>
> wrote:
>
>> +1
>>
>> ---
>> Luca Burgazzoli
>>
>>
>> On Mon, Jun 27, 2016 at 1:20 PM, Andrea Cosentino
>> < [hidden email]> wrote:
>> > +1.
>> >
>> > Tested against personal projects :-)
>> >
>> > Thanks Gregor!
>> > --
>> > Andrea Cosentino
>> > ----------------------------------
>> > Apache Camel PMC Member
>> > Apache Karaf Committer
>> > Apache Servicemix Committer
>> > Email: [hidden email]
>> > Twitter: @oscerd2
>> > Github: oscerd
>> >
>> >
>> >
>> > On Monday, June 27, 2016 8:42 AM, Gregor Zurowski <
>> [hidden email]> wrote:
>> > Hi Everyone:
>> >
>> > This is a vote to release Apache Camel 2.17.2, a new patch release
>> > that includes ~58 fixes and improvements.
>> >
>> > Release notes:
>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12335666&projectId=12311211>> >
>> > Staging repository:
>> > https://repository.apache.org/content/repositories/orgapachecamel-1051/>> >
>> > Tarballs:
>> https://repository.apache.org/content/repositories/orgapachecamel-1051/org/apache/camel/apache-camel/2.17.2/>> >
>> > Tag:
>> https://git-wip-us.apache.org/repos/asf?p=camel.git;a=tag;h=789a27911fc7787675b47298cd5a7d87448f60c8>> >
>> > Please test this release candidate and cast your vote.
>> > [ ] +1 Release the binary as Apache Camel 2.17.2
>> > [ ] -1 Veto the release (provide specific comments)
>> >
>> > The vote is open for at least 72 hours.
>> >
>> > Thanks,
>> > Gregor
>>
--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
|
|
|
ok, in that case +1, we will start using it soon (after a poc some moths
ago), and the fixes in came-kafka are very welcome.
On Mon, Jun 27, 2016 at 1:59 PM Claus Ibsen < [hidden email]> wrote:
> Hi
>
> No that is not a blocker. You can type in any string value you want, also
> -1.
>
> Its only the @UriParam that has a enum that suggests it must be
> "0,1,all". But there is no enforced validation on that.
>
>
>
> On Mon, Jun 27, 2016 at 1:48 PM, Andrea Cosentino
> < [hidden email]> wrote:
> > That's right.
> >
> > http://kafka.apache.org/documentation.html#producerconfigs> >
> > -1 is a valid value for acks.
> >
> > I'll commit a fix for this and maybe we'll need to cut the release again.
> >
> > My mistake. Sorry.
> > --
> > Andrea Cosentino
> > ----------------------------------
> > Apache Camel PMC Member
> > Apache Karaf Committer
> > Apache Servicemix Committer
> > Email: [hidden email]
> > Twitter: @oscerd2
> > Github: oscerd
> >
> >
> >
> > On Monday, June 27, 2016 1:41 PM, Gerard Klijs < [hidden email]>
> wrote:
> > -1 since https://issues.apache.org/jira/browse/CAMEL-9978 will make it
> > impossible to use "-1" as value for the acks, which is what we currently
> > use.
> >
> >
> > On Mon, Jun 27, 2016 at 1:21 PM Luca Burgazzoli < [hidden email]>
> > wrote:
> >
> >> +1
> >>
> >> ---
> >> Luca Burgazzoli
> >>
> >>
> >> On Mon, Jun 27, 2016 at 1:20 PM, Andrea Cosentino
> >> < [hidden email]> wrote:
> >> > +1.
> >> >
> >> > Tested against personal projects :-)
> >> >
> >> > Thanks Gregor!
> >> > --
> >> > Andrea Cosentino
> >> > ----------------------------------
> >> > Apache Camel PMC Member
> >> > Apache Karaf Committer
> >> > Apache Servicemix Committer
> >> > Email: [hidden email]
> >> > Twitter: @oscerd2
> >> > Github: oscerd
> >> >
> >> >
> >> >
> >> > On Monday, June 27, 2016 8:42 AM, Gregor Zurowski <
> >> [hidden email]> wrote:
> >> > Hi Everyone:
> >> >
> >> > This is a vote to release Apache Camel 2.17.2, a new patch release
> >> > that includes ~58 fixes and improvements.
> >> >
> >> > Release notes:
> >>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12335666&projectId=12311211> >> >
> >> > Staging repository:
> >> >
> https://repository.apache.org/content/repositories/orgapachecamel-1051/> >> >
> >> > Tarballs:
> >>
> https://repository.apache.org/content/repositories/orgapachecamel-1051/org/apache/camel/apache-camel/2.17.2/> >> >
> >> > Tag:
> >>
> https://git-wip-us.apache.org/repos/asf?p=camel.git;a=tag;h=789a27911fc7787675b47298cd5a7d87448f60c8> >> >
> >> > Please test this release candidate and cast your vote.
> >> > [ ] +1 Release the binary as Apache Camel 2.17.2
> >> > [ ] -1 Veto the release (provide specific comments)
> >> >
> >> > The vote is open for at least 72 hours.
> >> >
> >> > Thanks,
> >> > Gregor
> >>
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2>
|
|
|
Hi
Okay I tracked it down to when you do not provide a Content-Type when
using camel-restlet as a client to call a service. Then the default
content type is form based, but the message body may not be a form
structure.
Now camel-restlet then parses that differently on the server side
which causes this problem. But as a client if you provide the correct
Content-Type it works as expected.
This issue is not a blocker and I am reverting my -1.
I will continue testing the RC.
On Mon, Jun 27, 2016 at 2:14 PM, Claus Ibsen < [hidden email]> wrote:
--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
|
|
Administrator
|
+1
Christian
Am 27.06.2016 08:42 schrieb "Gregor Zurowski" < [hidden email]>:
|
|
|
Thanks Gregor for all the releases :-) --Andrea Cosentino ----------------------------------Apache Camel PMC MemberApache Karaf CommitterApache Servicemix CommitterEmail: [hidden email]: @oscerd2Github: oscerd
On Friday, July 1, 2016 9:29 AM, Claus Ibsen < [hidden email]> wrote:
Hi Gregor
I wonder if you have time to close this VOTE and get the release out?
On Mon, Jun 27, 2016 at 8:41 AM, Gregor Zurowski
< [hidden email]> wrote:
--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
|
|
|
Yes, I will get the release out shortly.
Thanks,
Gregor
On Jul 1, 2016 9:29 AM, "Claus Ibsen" < [hidden email]> wrote:
|
|
|
+1
Thanks,
Gregor
On Mon, Jun 27, 2016 at 8:41 AM, Gregor Zurowski
< [hidden email]> wrote:
|
12
|
Locked
