Re: Proposed changes for IVY-735 - Ability to specify timeouts

[jai . forums2013]

I think, that's actually a good idea. Defining the timeout constraints 
separately as named constraints and then referencing them would allow 
reusing these values on whichever resolvers want to:




 

read-timeout="">


...

 

 



 




-Jaikiran


On 14/07/17 10:33 PM, Gintautas Grigelionis wrote:

my €.02 : currently, all resolvers only have artifact patterns as child
elements, so I'd rather have the timeouts as attributes that might use
references, similarly to cache managers or latest strategies. That would
imply a global default that may be overridden individually.

Gintas

2017-07-14 13:34 GMT+02:00 :


https://issues.apache.org/jira/browse/IVY-735 is a feature request where
the users have asked for relevant timeouts while dealing with downloads. A
few weeks back we had a very brief discussion in an unrelated mail where it
was proposed that we allow configuring these timeout all the way from ivy
settings.

I have an initial proposal/attempt to implement this feature. The initial
changes are here in my personal repo[1]. It's a work in progress commit
which sets up the necessary interfaces and the flow to show what I have in
mind. Before proceeding further, I would like some inputs on whether this
looks fine. Here's a summary of what the changes are going to be:

- Each (dependency) resolver will have the ability to specify "timeout
constraints"[2].

- These timeout constraints will be specified while defining the resolver
in the ivy settings file. Imagine something like:

 

 

 

 

 

 

- Each of the resolver will then use these timeout constraints (if
specified) while resolving the module descriptor and downloading the
artifacts and dealing with those resources.

- The absence of the timeout constraints will let the resolvers behave the
way they do currently

The changes in the linked commit deprecate some APIs which weren't using
timeouts and are replaced by APIs which pass along (an optional)
TimeoutConstraints[2].

In summary, the change being proposed is that (dependency) resolvers have
the ability to specify these timeout constraints and then use them while
dealing with module descriptors and artifacts. One thing I have (to some
extent intentionally) left out is the ability to define a global Ivy
settings level or "all resolvers" level timeout-constraints. That's because
I'm not too sure if it adds much value instead of defining it at individual
resolver level. I'm however open to adding that support as well.

The linked commit currently doesn't have the necessary support for parsing
these additional XML elements, but if this whole approach looks fine, I
will take this further and make sure things work as expected.

[1] https://github.com/jaikiran/ant-ivy/commit/e501d9deca78db8b9
34f8a2710ebcfeaeb1456c8

[2] https://github.com/jaikiran/ant-ivy/commit/e501d9deca78db8b9
34f8a2710ebcfeaeb1456c8#diff-cd8ed454a52f4afa779574f5600a0ccb


-Jaikiran


-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org





-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org

[RESULT] [VOTE] IvyDE requirements

[Nicolas Lalevée]

Counting my own implicit +1 on both questions,

To make IvyDE require Java 7 as minimum, I count 3 binding +1, one non binding 
+1. The vote pass.
To make IvyDE require Eclipse 4.3 aka Eclipse Kepler as minimum, I count 3 
binding +1, one non binding +1. The vote pass.

Cheers,
Nicolas


> Le 11 juil. 2017 à 23:07, Nicolas Lalevée  a 
> écrit :
> 
> We have recently voted to make Ivy require Java 1.7. I think we should do it 
> for IvyDE too.
> Do you vote for it ?
> 
> 
> And while making the build of IvyDE work again nicely, I had to update its 
> dependencies. Mainly, the build requires Eclipse 4.3, aka Eclipse Kepler (it 
> is not the newest one as you can see [1]). So let’s make it official that we 
> require at least Eclipse 4.3.
> Do you vote for it ?
> 
> 
> Nicolas
> 
> [1] https://en.wikipedia.org/wiki/Eclipse_(software)#Releases 
> 
> 


-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org

Bug report for Ant [2017/07/16]

[bugzilla]

+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
| 5003|Opn|Blk|2001-11-21|exec task does not return after executed command f|
| 6606|Opn|Enh|2002-02-21|META-BUG problems with delegating classloaders|
| 7552|Opn|Cri|2002-03-28|Invalid task cache in targets |
| 7712|New|Enh|2002-04-03|Provide patternset support for VSSGET task?   |
| 8294|New|Enh|2002-04-19|: Allow  and  to app|
| 8866|New|Enh|2002-05-07|Signal handling in java task  |
| 8895|New|Enh|2002-05-08|ant and/or antcall should support forking |
| 8972|New|Enh|2002-05-10|allow property expansion in  property v|
| 8981|New|Enh|2002-05-10|Tar task command additional features  |
| 9294|New|Enh|2002-05-21|[PATCH] optional/j2ee/ServerDeploy OC4J Support   |
| 9995|Ass|Enh|2002-06-19|MKS Source Integrity tasks|
|10020|New|Enh|2002-06-19|'s dependency behaviour should be more con|
|10231|New|Enh|2002-06-25|Need access to current file in SQLExec|
|10283|New|Enh|2002-06-27|Add a destfile to the uptodate task   |
|10402|New|Enh|2002-07-02|adding the ability of html like whitespace preserv|
|3|New|Enh|2002-07-24|keytool task  |
|11560|Opn|Enh|2002-08-08|Taskdef does not apply reverseLoader policy on sta|
|12267|New|Enh|2002-09-03|Add ability to unzip into separate folders|
|12292|New|Enh|2002-09-04|[PATCH] enable  tag inside tar|
|12334|New|Enh|2002-09-05|REQUEST: Ant task doesn't allow attachment of a bu|
|12518|New|Enh|2002-09-11|Gunzip & BUnZip2 add filesets, patternsets, and ov|
|12765|New|Enh|2002-09-18|"rmdir" and "deltree" patches for ftp task enhance|
|12964|New|Enh|2002-09-24|ANTLR only takes one input file at a time |
|13047|Inf|Enh|2002-09-26|Support for  and  on O|
|13371|New|Enh|2002-10-07|[PATCH] Contributed new CvsExportDiff task|
|13847|New|Nor|2002-10-22|pvcs task: wrong option (-r) specified for get (sh|
|13934|New|Enh|2002-10-24|Translate task shouldn't load default locale prope|
|13939|New|Enh|2002-10-24|Translate task should have better key matching cap|
|14320|New|Enh|2002-11-06|copy fileset followsymlinks="false" does not copy |
|14393|New|Enh|2002-11-08|Support use of jndi within ant|
|14512|New|Enh|2002-11-13|Allow creating database connection similar to