[Vote] for CDI-616 resolution

classic Classic list List threaded Threaded
12 messages Options
Reply | Threaded
Open this post in threaded view
|

[Vote] for CDI-616 resolution

Antoine Sabot-Durand
Administrator
Hi all,

During yesterday(s meeting we discussed how to solve CDI-616 issue.
2 options are possible but we didn't find an agreement, so the best solution here would be to vote.

Options are:

a) Do nothing about injection in transient fields (todays behaviour) but add a clarification in the spec saying that using them is not supported.
b) Throw an exception at boot time if a transient field is an injection point.

To vote, just answer to this mail with the letter of your vote. Vote will last 72 hrs from the hour of this mail.

Thank you,

Antoine  

_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Emily Jiang
My vote is b.

Since it is not working in the current version, no functioning applications should rely on it. Throwing an exception is clearer to the developers.

Many thanks,
Emily
===========================
Emily Jiang
WebSphere Application Server, CDI Development Lead

 
MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
Phone:  +44 (0)1962 816278  Internal: 246278

Email: [hidden email]
Lotus Notes: Emily Jiang/UK/IBM@IBMGB




From:        Antoine Sabot-Durand <[hidden email]>
To:        cdi-dev <[hidden email]>,
Date:        03/08/2016 10:23
Subject:        [cdi-dev] [Vote] for CDI-616 resolution
Sent by:        [hidden email]




Hi all,

During yesterday(s meeting we discussed how to solve CDI-616 issue.
2 options are possible but we didn't find an agreement, so the best solution here would be to vote.

Options are:

a) Do nothing about injection in transient fields (todays behaviour) but add a clarification in the spec saying that using them is not supported.
b) Throw an exception at boot time if a transient field is an injection point.

To vote, just answer to this mail with the letter of your vote. Vote will last 72 hrs from the hour of this mail.

Thank you,

Antoine  _______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.

Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Romain Manni-Bucau
a - usable for wizards I think (init screen uses the field then other steps/after serialization dont touch it)


2016-08-03 11:38 GMT+02:00 Emily Jiang <[hidden email]>:
My vote is b.

Since it is not working in the current version, no functioning applications should rely on it. Throwing an exception is clearer to the developers.

Many thanks,
Emily
===========================
Emily Jiang
WebSphere Application Server, CDI Development Lead

 
MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
Phone:  +44 (0)1962 816278  Internal: 246278

Email: [hidden email]
Lotus Notes: Emily Jiang/UK/IBM@IBMGB




From:        Antoine Sabot-Durand <[hidden email]>
To:        cdi-dev <[hidden email]>,
Date:        03/08/2016 10:23
Subject:        [cdi-dev] [Vote] for CDI-616 resolution
Sent by:        [hidden email]




Hi all,

During yesterday(s meeting we discussed how to solve CDI-616 issue.
2 options are possible but we didn't find an agreement, so the best solution here would be to vote.

Options are:

a) Do nothing about injection in transient fields (todays behaviour) but add a clarification in the spec saying that using them is not supported.
b) Throw an exception at boot time if a transient field is an injection point.

To vote, just answer to this mail with the letter of your vote. Vote will last 72 hrs from the hour of this mail.

Thank you,

Antoine  _______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.

Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.


_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Matej Novotny
I vote for a)

----- Original Message -----

> From: "Romain Manni-Bucau" <[hidden email]>
> To: "Emily Jiang" <[hidden email]>
> Cc: "cdi-dev" <[hidden email]>
> Sent: Wednesday, August 3, 2016 11:42:32 AM
> Subject: Re: [cdi-dev] [Vote] for CDI-616 resolution
>
> a - usable for wizards I think (init screen uses the field then other
> steps/after serialization dont touch it)
>
>
> Romain Manni-Bucau
> @rmannibucau | Blog | Old Wordpress Blog | Github | LinkedIn | Tomitriber |
> JavaEE Factory
>
> 2016-08-03 11:38 GMT+02:00 Emily Jiang < [hidden email] > :
>
>
> My vote is b.
>
> Since it is not working in the current version, no functioning applications
> should rely on it. Throwing an exception is clearer to the developers.
>
> Many thanks,
> Emily
> ===========================
> Emily Jiang
> WebSphere Application Server, CDI Development Lead
>
> MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
> Phone: +44 (0)1962 816278 Internal: 246278
>
> Email: [hidden email]
> Lotus Notes: Emily Jiang/UK/IBM@IBMGB
>
>
>
>
> From: Antoine Sabot-Durand < [hidden email] >
> To: cdi-dev < [hidden email] >,
> Date: 03/08/2016 10:23
> Subject: [cdi-dev] [Vote] for CDI-616 resolution
> Sent by: [hidden email]
>
>
>
>
> Hi all,
>
> During yesterday(s meeting we discussed how to solve CDI-616 issue.
> 2 options are possible but we didn't find an agreement, so the best solution
> here would be to vote.
>
> Options are:
>
> a) Do nothing about injection in transient fields (todays behaviour) but add
> a clarification in the spec saying that using them is not supported.
> b) Throw an exception at boot time if a transient field is an injection
> point.
>
> To vote, just answer to this mail with the letter of your vote. Vote will
> last 72 hrs from the hour of this mail.
>
> Thank you,
>
> Antoine _______________________________________________
> cdi-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the code
> under the Apache License, Version 2 (
> http://www.apache.org/licenses/LICENSE-2.0.html ). For all other ideas
> provided on this list, the provider waives all patent and other intellectual
> property rights inherent in such information.
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
>
> _______________________________________________
> cdi-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the code
> under the Apache License, Version 2 (
> http://www.apache.org/licenses/LICENSE-2.0.html ). For all other ideas
> provided on this list, the provider waives all patent and other intellectual
> property rights inherent in such information.
>
>
> _______________________________________________
> cdi-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the code
> under the Apache License, Version 2
> (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas
> provided on this list, the provider waives all patent and other intellectual
> property rights inherent in such information.
_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Werner Keil-2
In reply to this post by Antoine Sabot-Durand
I'd say b.

Enforcing the correct behavior via API sounds better than a sentence in the spec. And makes it easier to keep implementations and applications compatible.

Werner



On Wed, Aug 3, 2016 at 12:03 PM, <[hidden email]> wrote:
Send cdi-dev mailing list submissions to
        [hidden email]

To subscribe or unsubscribe via the World Wide Web, visit
        https://lists.jboss.org/mailman/listinfo/cdi-dev
or, via email, send a message with subject or body 'help' to
        [hidden email]

You can reach the person managing the list at
        [hidden email]

When replying, please edit your Subject line so it is more specific
than "Re: Contents of cdi-dev digest..."


Today's Topics:

   1. [Vote] for CDI-616 resolution (Antoine Sabot-Durand)
   2. Re: [Vote] for CDI-616 resolution (Emily Jiang)
   3. Re: [Vote] for CDI-616 resolution (Romain Manni-Bucau)
   4. Re: [Vote] for CDI-616 resolution (Matej Novotny)


----------------------------------------------------------------------

Message: 1
Date: Wed, 03 Aug 2016 09:21:04 +0000
From: Antoine Sabot-Durand <[hidden email]>
Subject: [cdi-dev] [Vote] for CDI-616 resolution
To: cdi-dev <[hidden email]>
Message-ID:
        <[hidden email]>
Content-Type: text/plain; charset="utf-8"

Hi all,

During yesterday(s meeting we discussed how to solve CDI-616 issue.
2 options are possible but we didn't find an agreement, so the best
solution here would be to vote.

Options are:

a) Do nothing about injection in transient fields (todays behaviour) but
add a clarification in the spec saying that using them is not supported.
b) Throw an exception at boot time if a transient field is an injection
point.

To vote, just answer to this mail with the letter of your vote. Vote will
last 72 hrs from the hour of this mail.

Thank you,

Antoine
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/cdi-dev/attachments/20160803/16192536/attachment-0001.html

End of cdi-dev Digest, Vol 69, Issue 3
**************************************


_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Martin Kouba
In reply to this post by Emily Jiang
My vote is (b) - IF AND ONLY IF the definition error only applies to a
transient field injection point which resolves to a NORMAL SCOPED BEAN.
Because a contextual reference (client proxy) for such a bean is always
a passivation capable dependency. So there is no need to have it transient.

Otherwise my vote is (a).

Martin


Dne 3.8.2016 v 11:38 Emily Jiang napsal(a):

> My vote is b.
>
> Since it is not working in the current version, no functioning
> applications should rely on it. Throwing an exception is clearer to the
> developers.
>
> Many thanks,
> Emily
> ===========================
> Emily Jiang
> WebSphere Application Server, CDI Development Lead
>
> MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
> Phone:  +44 (0)1962 816278  Internal: 246278
>
> Email: [hidden email]
> Lotus Notes: Emily Jiang/UK/IBM@IBMGB
>
>
>
>
> From:        Antoine Sabot-Durand <[hidden email]>
> To:        cdi-dev <[hidden email]>,
> Date:        03/08/2016 10:23
> Subject:        [cdi-dev] [Vote] for CDI-616 resolution
> Sent by:        [hidden email]
> ------------------------------------------------------------------------
>
>
>
> Hi all,
>
> During yesterday(s meeting we discussed how to solve CDI-616 issue.
> 2 options are possible but we didn't find an agreement, so the best
> solution here would be to vote.
>
> Options are:
>
> a) Do nothing about injection in transient fields (todays behaviour) but
> add a clarification in the spec saying that using them is not supported.
> b) Throw an exception at boot time if a transient field is an injection
> point.
>
> To vote, just answer to this mail with the letter of your vote. Vote
> will last 72 hrs from the hour of this mail.
>
> Thank you,
>
> Antoine  _______________________________________________
> cdi-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the
> code under the Apache License, Version 2
> (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas
> provided on this list, the provider waives all patent and other
> intellectual property rights inherent in such information.
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
>
>
> _______________________________________________
> cdi-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
>

--
Martin Kouba
Software Engineer
Red Hat, Czech Republic
_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

John Ament
In reply to this post by Antoine Sabot-Durand

My vote is (a).  The current state is that this kind of works, and we would be breaking existing apps by throwing an exception.


From: [hidden email] <[hidden email]> on behalf of Antoine Sabot-Durand <[hidden email]>
Sent: Wednesday, August 3, 2016 5:21:04 AM
To: cdi-dev
Subject: [cdi-dev] [Vote] for CDI-616 resolution
 
Hi all,

During yesterday(s meeting we discussed how to solve CDI-616 issue.
2 options are possible but we didn't find an agreement, so the best solution here would be to vote.

Options are:

a) Do nothing about injection in transient fields (todays behaviour) but add a clarification in the spec saying that using them is not supported.
b) Throw an exception at boot time if a transient field is an injection point.

To vote, just answer to this mail with the letter of your vote. Vote will last 72 hrs from the hour of this mail.

Thank you,

Antoine  

NOTICE: This e-mail message and any attachments may contain confidential, proprietary, and/or privileged information which should be treated accordingly. If you are not the intended recipient, please notify the sender immediately by return e-mail, delete this message, and destroy all physical and electronic copies. Thank you.
_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Fw: [Vote] for CDI-616 resolution

Mark Struberg
In reply to this post by Antoine Sabot-Durand
a.)




> On Wednesday, 3 August 2016, 13:04, Mark Struberg <[hidden email]> wrote:
> > a.)
>
>
>
> txs and LieGrue,
> strub
>
>
> On Wednesday, 3 August 2016, 11:22, Antoine Sabot-Durand
> <[hidden email]> wrote:
>
>> Hi all,
>>
>>
>> During yesterday(s meeting we discussed how to solve CDI-616 issue.
>> 2 options are possible but we didn't find an agreement, so the best
> solution here would be to vote.
>>
>>
>> Options are:
>>
>>
>> a) Do nothing about injection in transient fields (todays behaviour) but add
> a clarification in the spec saying that using them is not supported.
>> b) Throw an exception at boot time if a transient field is an injection
> point.
>>
>>
>> To vote, just answer to this mail with the letter of your vote. Vote will
> last 72 hrs from the hour of this mail.
>>
>>
>> Thank you,
>>
>>
>> Antoine  
>> _______________________________________________
>> cdi-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/cdi-dev
>>
>> Note that for all code provided on this list, the provider licenses the code
> under the Apache License, Version 2
> (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided
> on this list, the provider waives all patent and other intellectual property
> rights inherent in such information.
>>
>>
>
_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Stephan Knitelius
In reply to this post by John Ament
(b) - apps defining transient inject points in passivatable scopes are broken to start with.     

On Wed, 3 Aug 2016 at 13:05 John Ament <[hidden email]> wrote:

My vote is (a).  The current state is that this kind of works, and we would be breaking existing apps by throwing an exception.


From: [hidden email] <[hidden email]> on behalf of Antoine Sabot-Durand <[hidden email]>
Sent: Wednesday, August 3, 2016 5:21:04 AM
To: cdi-dev

Subject: [cdi-dev] [Vote] for CDI-616 resolution
Hi all,

During yesterday(s meeting we discussed how to solve CDI-616 issue.
2 options are possible but we didn't find an agreement, so the best solution here would be to vote.

Options are:

a) Do nothing about injection in transient fields (todays behaviour) but add a clarification in the spec saying that using them is not supported.
b) Throw an exception at boot time if a transient field is an injection point.

To vote, just answer to this mail with the letter of your vote. Vote will last 72 hrs from the hour of this mail.

Thank you,

Antoine  

NOTICE: This e-mail message and any attachments may contain confidential, proprietary, and/or privileged information which should be treated accordingly. If you are not the intended recipient, please notify the sender immediately by return e-mail, delete this message, and destroy all physical and electronic copies. Thank you.
_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.

_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Emily Jiang
In reply to this post by Martin Kouba
I think there are 4a and 4b
I'm happy for b to be reworded to

b) A definition error will be thrown if the injection point is transient and resolves to a normal scoped bean.


Many thanks,
Emily
===========================
Emily Jiang
WebSphere Application Server, CDI Development Lead

 
MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
Phone:  +44 (0)1962 816278  Internal: 246278

Email: [hidden email]
Lotus Notes: Emily Jiang/UK/IBM@IBMGB




From:        Martin Kouba <[hidden email]>
To:        [hidden email],
Date:        03/08/2016 11:22
Subject:        Re: [cdi-dev] [Vote] for CDI-616 resolution
Sent by:        [hidden email]




My vote is (b) - IF AND ONLY IF the definition error only applies to a
transient field injection point which resolves to a NORMAL SCOPED BEAN.
Because a contextual reference (client proxy) for such a bean is always
a passivation capable dependency. So there is no need to have it transient.

Otherwise my vote is (a).

Martin


Dne 3.8.2016 v 11:38 Emily Jiang napsal(a):
> My vote is b.
>
> Since it is not working in the current version, no functioning
> applications should rely on it. Throwing an exception is clearer to the
> developers.
>
> Many thanks,
> Emily
> ===========================
> Emily Jiang
> WebSphere Application Server, CDI Development Lead
>
> MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
> Phone:  +44 (0)1962 816278  Internal: 246278
>
> Email: [hidden email]
> Lotus Notes: Emily Jiang/UK/IBM@IBMGB
>
>
>
>
> From:        Antoine Sabot-Durand <[hidden email]>
> To:        cdi-dev <[hidden email]>,
> Date:        03/08/2016 10:23
> Subject:        [cdi-dev] [Vote] for CDI-616 resolution
> Sent by:        [hidden email]
> ------------------------------------------------------------------------
>
>
>
> Hi all,
>
> During yesterday(s meeting we discussed how to solve CDI-616 issue.
> 2 options are possible but we didn't find an agreement, so the best
> solution here would be to vote.
>
> Options are:
>
> a) Do nothing about injection in transient fields (todays behaviour) but
> add a clarification in the spec saying that using them is not supported.
> b) Throw an exception at boot time if a transient field is an injection
> point.
>
> To vote, just answer to this mail with the letter of your vote. Vote
> will last 72 hrs from the hour of this mail.
>
> Thank you,
>
> Antoine  _______________________________________________
> cdi-dev mailing list
> [hidden email]
>
https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the
> code under the Apache License, Version 2
> (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas
> provided on this list, the provider waives all patent and other
> intellectual property rights inherent in such information.
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
>
>
> _______________________________________________
> cdi-dev mailing list
> [hidden email]
>
https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
>

--
Martin Kouba
Software Engineer
Red Hat, Czech Republic
_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.



Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Thorben Janssen
Enforcing the right usage would be great, but it would also break existing applications.

I, therefore, vote: (a)

Developers who introduced this to their code most likely handle the existing behaviour in some way and I don't see a reason to break that.


2016-08-03 14:55 GMT+02:00 Emily Jiang <[hidden email]>:
I think there are 4a and 4b
I'm happy for b to be reworded to

b) A definition error will be thrown if the injection point is transient and resolves to a normal scoped bean.


Many thanks,
Emily
===========================
Emily Jiang
WebSphere Application Server, CDI Development Lead

 
MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
Phone:  <a href="tel:%2B44%20%280%291962%20816278" value="+441962816278" target="_blank">+44 (0)1962 816278  Internal: 246278

Email: [hidden email]
Lotus Notes: Emily Jiang/UK/IBM@IBMGB




From:        Martin Kouba <[hidden email]>
To:        [hidden email],
Date:        03/08/2016 11:22
Subject:        Re: [cdi-dev] [Vote] for CDI-616 resolution
Sent by:        [hidden email]




My vote is (b) - IF AND ONLY IF the definition error only applies to a
transient field injection point which resolves to a NORMAL SCOPED BEAN.
Because a contextual reference (client proxy) for such a bean is always
a passivation capable dependency. So there is no need to have it transient.

Otherwise my vote is (a).

Martin


Dne 3.8.2016 v 11:38 Emily Jiang napsal(a):
> My vote is b.
>
> Since it is not working in the current version, no functioning
> applications should rely on it. Throwing an exception is clearer to the
> developers.
>
> Many thanks,
> Emily
> ===========================
> Emily Jiang
> WebSphere Application Server, CDI Development Lead
>
> MP 211, DE3A20, Winchester, Hampshire, England, SO21 2JN
> Phone:  <a href="tel:%2B44%20%280%291962%20816278" value="+441962816278" target="_blank">+44 (0)1962 816278  Internal: 246278
>
> Email: [hidden email]
> Lotus Notes: Emily Jiang/UK/IBM@IBMGB
>
>
>
>
> From:        Antoine Sabot-Durand <[hidden email]>
> To:        cdi-dev <[hidden email]>,
> Date:        03/08/2016 10:23
> Subject:        [cdi-dev] [Vote] for CDI-616 resolution
> Sent by:        [hidden email]
> ------------------------------------------------------------------------
>
>
>
> Hi all,
>
> During yesterday(s meeting we discussed how to solve CDI-616 issue.
> 2 options are possible but we didn't find an agreement, so the best
> solution here would be to vote.
>
> Options are:
>
> a) Do nothing about injection in transient fields (todays behaviour) but
> add a clarification in the spec saying that using them is not supported.
> b) Throw an exception at boot time if a transient field is an injection
> point.
>
> To vote, just answer to this mail with the letter of your vote. Vote
> will last 72 hrs from the hour of this mail.
>
> Thank you,
>
> Antoine  _______________________________________________
> cdi-dev mailing list
> [hidden email]
>
https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the
> code under the Apache License, Version 2
> (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas
> provided on this list, the provider waives all patent and other
> intellectual property rights inherent in such information.
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
>
>
> _______________________________________________
> cdi-dev mailing list
> [hidden email]
>
https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
>

--
Martin Kouba
Software Engineer
Red Hat, Czech Republic
_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (
http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.



Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.


_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.
Reply | Threaded
Open this post in threaded view
|

Re: [Vote] for CDI-616 resolution

Tomas Remes
In reply to this post by Antoine Sabot-Durand

a)

----- Original Message -----
From: "Antoine Sabot-Durand" <[hidden email]>
To: "cdi-dev" <[hidden email]>
Sent: Wednesday, August 3, 2016 11:21:04 AM
Subject: [cdi-dev] [Vote] for CDI-616 resolution

Hi all,

During yesterday(s meeting we discussed how to solve CDI-616 issue.
2 options are possible but we didn't find an agreement, so the best solution here would be to vote.

Options are:

a) Do nothing about injection in transient fields (todays behaviour) but add a clarification in the spec saying that using them is not supported.
b) Throw an exception at boot time if a transient field is an injection point.

To vote, just answer to this mail with the letter of your vote. Vote will last 72 hrs from the hour of this mail.

Thank you,

Antoine

_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.

--
Tomas Remes


_______________________________________________
cdi-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/cdi-dev

Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.