[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13135736#comment-13135736
]
Simone Tripodi commented on OGNL-20:
good catch Daniel!!!
> Performance
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13135508#comment-13135508
]
Maurizio Cucchiara commented on OGNL-20:
During the merge I had a lot of problem with
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13135490#comment-13135490
]
Daniel Pitts commented on OGNL-20:
--
Maurizio: I have a suggestion for next time; Merge trunk
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133952#comment-13133952
]
Simone Tripodi commented on OGNL-20:
ouch, you risked you own skin! :)
>
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133910#comment-13133910
]
Maurizio Cucchiara commented on OGNL-20:
Not exactly.
I was talking about the merge r
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133894#comment-13133894
]
Simone Tripodi commented on OGNL-20:
It was anyway saved in the branch, or not? ;)
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133889#comment-13133889
]
Maurizio Cucchiara commented on OGNL-20:
I'm sorry for the missing announce. In my de
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133846#comment-13133846
]
Simone Tripodi commented on OGNL-20:
Hola Mau,
after all the effort you put on this issue
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13133777#comment-13133777
]
Hudson commented on OGNL-20:
Integrated in ognl #147 (See [https://builds.apache.org/job/ognl/147
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13128550#comment-13128550
]
Maurizio Cucchiara commented on OGNL-20:
OK guys,
my green bar told me that I'm almos
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13128104#comment-13128104
]
Maurizio Cucchiara commented on OGNL-20:
Hi Daniel,
Thanks for your preciuos feedback
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13127818#comment-13127818
]
Daniel Pitts commented on OGNL-20:
--
{quote}
Unfortunately, not everything turns out as it sh
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13126518#comment-13126518
]
Simone Tripodi commented on OGNL-20:
Hi Mau,
How can I check which are the differences be
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13126472#comment-13126472
]
Maurizio Cucchiara commented on OGNL-20:
Hi Daniel,
{quote}
First suggestion is ditch
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13125984#comment-13125984
]
Daniel Pitts commented on OGNL-20:
--
I've just now gone over the branch, and have a couple of
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13125661#comment-13125661
]
Maurizio Cucchiara commented on OGNL-20:
Hi Daniel,
I know that, infact I am not goin
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13125544#comment-13125544
]
Daniel Pitts commented on OGNL-20:
--
Maurizio, my patch was actually just for benchmarking th
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13124162#comment-13124162
]
Maurizio Cucchiara commented on OGNL-20:
Ok guys,
here we go, I have just committed t
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13124136#comment-13124136
]
Maurizio Cucchiara commented on OGNL-20:
Ok,
it is not simple to explain, I'm going t
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13123969#comment-13123969
]
Simone Tripodi commented on OGNL-20:
Hi Mau,
I need some clarifications because I am a li
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13123959#comment-13123959
]
Maurizio Cucchiara commented on OGNL-20:
Hi guys,
I'm trying to apply a revisited ver
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13109631#comment-13109631
]
Daniel Pitts commented on OGNL-20:
--
Adrian, All I was saying is that I believe this discussi
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13109372#comment-13109372
]
Adrian Crum commented on OGNL-20:
-
Daniel,
Keep in mind that somewhere down the road you wil
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13108792#comment-13108792
]
Daniel Pitts commented on OGNL-20:
--
I understand that the DCL is a broken pattern, what I su
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13108221#comment-13108221
]
Adrian Crum commented on OGNL-20:
-
That pattern (DCL) should not be used:
http://www.cs.umd.
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13108188#comment-13108188
]
Daniel Pitts commented on OGNL-20:
--
I have read many times Java Concurrency in Practice, and
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13108134#comment-13108134
]
Adrian Crum commented on OGNL-20:
-
Daniel: The DCL design pattern does not enforce a singleto
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13108041#comment-13108041
]
Daniel Pitts commented on OGNL-20:
--
Thanks for your feedback Adrian, I appreciate it.
The c
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13108027#comment-13108027
]
Adrian Crum commented on OGNL-20:
-
Daniel: It appears that you are trying to implement a gene
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13108014#comment-13108014
]
Daniel Pitts commented on OGNL-20:
--
Adrian: You mean the patch I attached? I'm pretty sure n
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107973#comment-13107973
]
Maurizio Cucchiara commented on OGNL-20:
I am not sure about it, but I think that CHM
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107962#comment-13107962
]
Adrian Crum commented on OGNL-20:
-
Two of the approaches in the patch will have concurrency i
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107929#comment-13107929
]
Simone Tripodi commented on OGNL-20:
Cool, thanks a lot for the effort Mau and congrats f
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107919#comment-13107919
]
Maurizio Cucchiara commented on OGNL-20:
I have been very busy last week, I just fini
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13107898#comment-13107898
]
Simone Tripodi commented on OGNL-20:
I personally would like to postpone eventual refacto
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13101057#comment-13101057
]
Simone Tripodi commented on OGNL-20:
Experienced users like you are lifeblood for OGNL, y
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13101040#comment-13101040
]
Daniel Pitts commented on OGNL-20:
--
You explained it well enough, I was just trying to justi
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13101013#comment-13101013
]
Simone Tripodi commented on OGNL-20:
Sorry, I just figure out that I explained so bad to
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13101002#comment-13101002
]
Daniel Pitts commented on OGNL-20:
--
Simone, I never meant to imply it was Ognl by anyone oth
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13099181#comment-13099181
]
Simone Tripodi commented on OGNL-20:
Daniel, the original version of OGNL was developed b
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13099054#comment-13099054
]
Daniel Pitts commented on OGNL-20:
--
I was looking through OgnlRuntime.java, and it looks lik
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098778#comment-13098778
]
Maurizio Cucchiara commented on OGNL-20:
Good Morning Julien,
It would be very very a
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098779#comment-13098779
]
Maurizio Cucchiara commented on OGNL-20:
Good Morning Julien,
It would be very very a
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098773#comment-13098773
]
Julien Aymé commented on OGNL-20:
-
Hi all,
I will provide both a small benchmark test and a
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098677#comment-13098677
]
Simone Tripodi commented on OGNL-20:
the OGNL JVM target is 1.5, see
[pom.xml|https://sv
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098362#comment-13098362
]
Daniel Pitts commented on OGNL-20:
--
Do keep in mind that the ConcurrentHashMap is new in Jav
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098161#comment-13098161
]
Simone Tripodi commented on OGNL-20:
OK for that case you convinced me, thanks :)
Looking
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098134#comment-13098134
]
Maurizio Cucchiara commented on OGNL-20:
Julien,
it looks like one of us is able to r
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098110#comment-13098110
]
Julien Aymé commented on OGNL-20:
-
Simone,
The complexity of lock/unlock operation is O(1) I
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098108#comment-13098108
]
Maurizio Cucchiara commented on OGNL-20:
Hi Simone,
to answer your questions...
{quot
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098073#comment-13098073
]
Simone Tripodi commented on OGNL-20:
Julien,
Which complexity has lock/unlock objects? I'
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098040#comment-13098040
]
Julien Aymé commented on OGNL-20:
-
Hi Simone !
You got it right for the main part, except th
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098029#comment-13098029
]
Simone Tripodi commented on OGNL-20:
Salut Julien!
of course I understand the reasons - e
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098009#comment-13098009
]
Julien Aymé commented on OGNL-20:
-
@Maurizio:
Yes, putIfAbsent is a good idea !
@Simone,
you
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13097998#comment-13097998
]
Simone Tripodi commented on OGNL-20:
+1 to {{ReentrantReadWriteLocks}}.
IMHO
{code}
Map
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13097868#comment-13097868
]
Adrian Crum commented on OGNL-20:
-
I highly recommend reading Java Concurrency In Practice
(
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13097814#comment-13097814
]
Maurizio Cucchiara commented on OGNL-20:
Hi Julien,
that was my original thought, fu
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13097809#comment-13097809
]
Julien Aymé commented on OGNL-20:
-
Note that you also could use ConcurrentHashMap instead of
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13097795#comment-13097795
]
Maurizio Cucchiara commented on OGNL-20:
Hi Olivier,
be aware that there is already:
[
https://issues.apache.org/jira/browse/OGNL-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13097791#comment-13097791
]
Olivier Lamy commented on OGNL-20:
--
good idea ! I will try a patch or a branch with that.
>
60 matches
Mail list logo
