You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by Apache Hudson Server <hu...@hudson.apache.org> on 2010/08/29 22:50:22 UTC

Build failed in Hudson: Shindig #1792

See <https://hudson.apache.org/hudson/job/Shindig/1792/>

------------------------------------------
[...truncated 3096 lines...]
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__ug_Arab.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__bg_BG.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sa.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ml_IN.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ar_DZ.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__uz_Cyrl_UZ.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sr_ME.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__hi.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__ee_GH.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sl_SI.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sk_SK.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ha_GH.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__se.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__es_MX.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__gez_ER.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__gu.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__cch_NG.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__es_EC.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__es_UY.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__en_NA.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__kl_GL.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ku_TR.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__wo_SN.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__id.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__es_DO.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ny_MW.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ha_Latn_NG.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__om_KE.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sr_YU.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__en_GU.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ku_Latn.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__mk_MK.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__mr_IN.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__az_AZ.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__bn_BD.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__kl.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ss_SZ.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__da_DK.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__pa_Guru.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__et.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__hu_HU.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__en_UM.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ti_ET.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sl.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__ha_SD.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__nl.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ar_BH.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__mt.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__af_ZA.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__fi_FI.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__uz_UZ.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__ig_NG.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__en_Shaw.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__se_FI.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__gez.js
A         features/src/main/javascript/features/i18n/data/DateTimeConstants__bo_IN.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__vi_VN.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__az_Latn.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__pt.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__fr_MC.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__bg.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__ar_OM.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__sr_RS.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__kfo_CI.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__rw_RW.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__nb_NO.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ug_CN.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__zh_Hant.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__mn_Cyrl.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sr_Latn_BA.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__sq_AL.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__aa_ER.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__as.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sr_CS.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__fil.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__gsw_CH.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__yo_NG.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__dv_MV.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__kpe.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__lo.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__is.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ru_UA.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__kw.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__gv_GB.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__pa_Arab.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__yo.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__to.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__syr_SY.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__sw.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__pa_PK.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__en_AS.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__es_CR.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__kcg_NG.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__bn.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__kpe_GN.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__nb.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__bg_BG.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__en.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__az_Cyrl.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__it_IT.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ha_Arab_SD.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__dv.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ku_Arab.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__zh_Hans_CN.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__az.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__sr_ME.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__kpe_LR.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__ee_GH.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__sl_SI.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__mn.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__sk_SK.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__ug_Arab_CN.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__or.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__lv.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__gv.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__uz_Latn.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__dz.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__en_HK.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__da.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__es_PA.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__ps_AF.js
AU        features/src/main/javascript/features/i18n/data/NumberFormatConstants__es_UY.js
AU        features/src/main/javascript/features/i18n/data/DateTimeConstants__en_SG.js
AU        features/src/main/javascript/features/i18n/feature.xml
AU        features/src/main/javascript/features/i18n/numberformat.js
AU        features/src/main/javascript/features/i18n/numberformattest.js
AU        features/src/main/javascript/features/i18n/currencycodemap.js
A         features/src/main/javascript/features/container
A         features/src/main/javascript/features/container/gadget_holder.js
A         features/src/main/javascript/features/container/util.js
A         features/src/main/javascript/features/container/feature.xml
A         features/src/main/javascript/features/container/gadget_site.js
A         features/src/main/javascript/features/container/service.js
A         features/src/main/javascript/features/container/container.js
A         features/src/main/javascript/features/container/init.js
A         features/src/main/javascript/features/globals
A         features/src/main/javascript/features/globals/feature.xml
A         features/src/main/javascript/features/globals/globals.js
A         features/src/main/javascript/features/security-token
A         features/src/main/javascript/features/security-token/feature.xml
A         features/src/main/javascript/features/opensocial-data-context
AU        features/src/main/javascript/features/opensocial-data-context/datacontext.js
AU        features/src/main/javascript/features/opensocial-data-context/feature.xml
A         features/src/main/javascript/features/opensocial-data-context/taming.js
A         features/src/main/javascript/features/swfobject
AU        features/src/main/javascript/features/swfobject/swfobject.js
AU        features/src/main/javascript/features/swfobject/swfobject.opt.js
AU        features/src/main/javascript/features/swfobject/feature.xml
A         features/src/main/javascript/features/shindig.xhrwrapper
A         features/src/main/javascript/features/shindig.xhrwrapper/feature.xml
A         features/src/main/javascript/features/shindig.xhrwrapper/xhrwrapper.js
A         features/src/main/javascript/features/analytics
AU        features/src/main/javascript/features/analytics/feature.xml
A         features/src/main/javascript/features/opensocial-1.0
A         features/src/main/javascript/features/opensocial-1.0/feature.xml
A         features/src/main/javascript/features/core.auth
A         features/src/main/javascript/features/tabs
AU        features/src/main/javascript/features/tabs/tabs.js
AU        features/src/main/javascript/features/tabs/feature.xml
A         features/src/main/javascript/features/tabs/taming.js
A         features/src/main/javascript/features/dynamic-height
AU        features/src/main/javascript/features/dynamic-height/dynamic-height.js
AU        features/src/main/javascript/features/dynamic-height/feature.xml
A         features/src/main/javascript/features/dynamic-height/taming.js
A         features/src/main/javascript/features/opensocial-0.6
AU        features/src/main/javascript/features/opensocial-0.6/opensocial6to7.js
AU        features/src/main/javascript/features/opensocial-0.6/feature.xml
A         features/src/main/javascript/features/flash
AU        features/src/main/javascript/features/flash/flash.js
AU        features/src/main/javascript/features/flash/feature.xml
A         features/src/main/javascript/features/flash/taming.js
A         features/src/main/javascript/features/settitle
AU        features/src/main/javascript/features/settitle/settitle.js
AU        features/src/main/javascript/features/settitle/feature.xml
A         features/src/main/javascript/features/settitle/taming.js
A         features/src/main/javascript/features/views
AU        features/src/main/javascript/features/views/views.js
AU        features/src/main/javascript/features/views/feature.xml
A         features/src/main/javascript/features/views/taming.js
A         features/src/main/javascript/features/opensocial-0.7
AU        features/src/main/javascript/features/opensocial-0.7/opensocial7to8.js
AU        features/src/main/javascript/features/opensocial-0.7/feature.xml
A         features/src/main/javascript/features/caja
AU        features/src/main/javascript/features/caja/caja.js
AU        features/src/main/javascript/features/caja/feature.xml
A         features/src/main/javascript/features/caja/taming.js
A         features/src/main/javascript/features/xmlutil
AU        features/src/main/javascript/features/xmlutil/xmlutil.js
AU        features/src/main/javascript/features/xmlutil/feature.xml
A         features/src/main/javascript/features/opensocial-0.8
AU        features/src/main/javascript/features/opensocial-0.8/feature.xml
AU        features/src/main/javascript/features/opensocial-0.8/opensocial8to9.js
A         features/src/main/javascript/features/opensocial-0.9
AU        features/src/main/javascript/features/opensocial-0.9/feature.xml
A         features/src/main/javascript/features/core.json
AU        features/src/main/javascript/features/core.json/json.js
A         features/src/main/javascript/features/core.json/feature.xml
A         features/src/main/javascript/features/core.json/taming.js
A         features/src/main/javascript/features/pubsub
AU        features/src/main/javascript/features/pubsub/pubsub-router.js
AU        features/src/main/javascript/features/pubsub/feature.xml
A         features/src/main/javascript/features/pubsub/taming.js
AU        features/src/main/javascript/features/pubsub/pubsub.js
A         features/src/main/javascript/features/minimessage
AU        features/src/main/javascript/features/minimessage/feature.xml
A         features/src/main/javascript/features/minimessage/taming.js
AU        features/src/main/javascript/features/minimessage/minimessage.js
A         features/src/main/javascript/features/shindig.container-1.0
A         features/src/main/javascript/features/shindig.container-1.0/feature.xml
A         features/src/main/javascript/features/xhrwrapper
A         features/src/main/javascript/features/xhrwrapper/feature.xml
A         features/src/main/javascript/features/opensocial-data
AU        features/src/main/javascript/features/opensocial-data/data.js
AU        features/src/main/javascript/features/opensocial-data/feature.xml
A         features/src/main/javascript/features/dynamic-height.util
A         features/src/main/javascript/features/dynamic-height.util/dynamic-height-util.js
A         features/src/main/javascript/features/dynamic-height.util/feature.xml
A         features/src/main/javascript/features/opensocial-jsonrpc
AU        features/src/main/javascript/features/opensocial-jsonrpc/feature.xml
AU        features/src/main/javascript/features/opensocial-jsonrpc/jsonrpccontainer.js
A         features/src/main/javascript/features/jsondom
A         features/src/main/javascript/features/jsondom/jsondom.js
A         features/src/main/javascript/features/jsondom/feature.xml
A         features/src/main/javascript/features/osml
A         features/src/main/javascript/features/osml/feature.xml
A         features/src/main/javascript/features/osapi
AU        features/src/main/javascript/features/osapi/peoplehelpers.js
A         features/src/main/javascript/features/osapi/jsonrpctransport.js
AU        features/src/main/javascript/features/osapi/osapi.js
AU        features/src/main/javascript/features/osapi/feature.xml
A         features/src/main/javascript/features/osapi/taming.js
A         features/src/main/javascript/features/osapi/gadgetsrpctransport.js
AU        features/src/main/javascript/features/osapi/batch.js
A         features/src/main/javascript/features/opensocial-current
AU        features/src/main/javascript/features/opensocial-current/feature.xml
A         features/src/site
AU        features/src/site/site.xml
A         features/bin
AU        features/bin/runner.sh
AU        features/bin/README
AU        features/pom.xml
AU        features/README
AU        BUILD-JAVA
 U        .
At revision 990649
no revision recorded for http://svn.apache.org/repos/asf/shindig/trunk in the previous build
Parsing POMs
ERROR: No classworlds*.jar found in /home/hudson/tools/maven/apache-maven-2.2.1 -- Is this a valid maven2 directory?
[WARNINGS] Skipping publisher since build result is FAILURE


Re: Hudson build is back to normal : Shindig #1793

Posted by Mat Mannion <M....@warwick.ac.uk>.
A SortedMap approach wouldn't preserve insertion order, but would
instead re-order parameters on the fly with the comparator provided. A
LinkedHashMap (or LinkedMultimap, which seems like a better solution
as you suggest) preserves insertion order, so this would be the
correct way to go if we're looking to preserve the parameter order.

Mat

On 31 August 2010 10:23, Gagandeep singh <ga...@gmail.com> wrote:
> I have a doubt. A fellow colleague tells me that we should retain the order
> in which the keys are inserted into the query params map, because some ppl
> do funny stuff like their own begin and end parameters and mark useful
> parameters in between.
>
> I was wondering if we should go with a SortedMap approach for query params
> instead. I mean do we really need to bother about query param P1 added
> before query param P2 ? Also, tomorrow if some1 refactores some code, the
> tests should not fail because the order of insertion has changed.
>
> As a side note, i think Google uses LinkedListMultimap for query params.
> Isn't that better than LinkedHashMap<String, List<String>> ?
>
> Thoughts ?
>
>
>
> On Mon, Aug 30, 2010 at 1:31 PM, Gagandeep singh <ga...@gmail.com>wrote:
>
>> Nice debugging.
>>
>>
>> On Mon, Aug 30, 2010 at 7:42 AM, Paul Lindner <pl...@linkedin.com>wrote:
>>
>>> The test output showed that the problem was incorrectly ordered query
>>> params.  I validated that the problem occurs only on Java 1.5.  Therefore
>>> it
>>> had to be a HashMap iteration problem somewhere.  So I poked around
>>> UriBuilder and found one remaining use of HashMap and replaced it with
>>> LinkedHashMap.
>>>
>>> On Sun, Aug 29, 2010 at 6:54 PM, Gagandeep singh <gagan.goku@gmail.com
>>> >wrote:
>>>
>>> > Could you also tell us how you found and fixed the problem ?
>>> >
>>> > On Mon, Aug 30, 2010 at 7:24 AM, Gagandeep singh <gagan.goku@gmail.com
>>> > >wrote:
>>> >
>>> > > Thanks for fixing this Paul.
>>> > >
>>> > >
>>> > > On Mon, Aug 30, 2010 at 2:46 AM, Apache Hudson Server <
>>> > > hudson@hudson.apache.org> wrote:
>>> > >
>>> > >> See <https://hudson.apache.org/hudson/job/Shindig/1793/>
>>> > >>
>>> > >>
>>> > >>
>>> > >
>>> >
>>>
>>>
>>>
>>> --
>>> Paul Lindner -- plindner@linkedin.com -- linkedin.com/in/plindner
>>>
>>
>>
>



-- 
Mat Mannion
Web Developer
IT Services
University of Warwick
Coventry
CV4 7AL

Tel: 024 765 74433
Email: M.Mannion@warwick.ac.uk

Re: Hudson build is back to normal : Shindig #1793

Posted by Paul Lindner <pl...@linkedin.com>.
Our current UriBuilder has nice reflective properties like this:

String foo = "http://foo.com/a?z=1&a=2";

UriBuilder.parse(foo).toString().equals(foo);

I think this behavior is worth preserving

I would like to see a Multimap implementation of query params.  Note that
ImmutableListMultimaps also retain their insertion order.


On Tue, Aug 31, 2010 at 2:23 AM, Gagandeep singh <ga...@gmail.com>wrote:

> I have a doubt. A fellow colleague tells me that we should retain the order
> in which the keys are inserted into the query params map, because some ppl
> do funny stuff like their own begin and end parameters and mark useful
> parameters in between.
>
> I was wondering if we should go with a SortedMap approach for query params
> instead. I mean do we really need to bother about query param P1 added
> before query param P2 ? Also, tomorrow if some1 refactores some code, the
> tests should not fail because the order of insertion has changed.
>
> As a side note, i think Google uses LinkedListMultimap for query params.
> Isn't that better than LinkedHashMap<String, List<String>> ?
>
> Thoughts ?
>
>
>
> On Mon, Aug 30, 2010 at 1:31 PM, Gagandeep singh <gagan.goku@gmail.com
> >wrote:
>
> > Nice debugging.
> >
> >
> > On Mon, Aug 30, 2010 at 7:42 AM, Paul Lindner <plindner@linkedin.com
> >wrote:
> >
> >> The test output showed that the problem was incorrectly ordered query
> >> params.  I validated that the problem occurs only on Java 1.5.
>  Therefore
> >> it
> >> had to be a HashMap iteration problem somewhere.  So I poked around
> >> UriBuilder and found one remaining use of HashMap and replaced it with
> >> LinkedHashMap.
> >>
> >> On Sun, Aug 29, 2010 at 6:54 PM, Gagandeep singh <gagan.goku@gmail.com
> >> >wrote:
> >>
> >> > Could you also tell us how you found and fixed the problem ?
> >> >
> >> > On Mon, Aug 30, 2010 at 7:24 AM, Gagandeep singh <
> gagan.goku@gmail.com
> >> > >wrote:
> >> >
> >> > > Thanks for fixing this Paul.
> >> > >
> >> > >
> >> > > On Mon, Aug 30, 2010 at 2:46 AM, Apache Hudson Server <
> >> > > hudson@hudson.apache.org> wrote:
> >> > >
> >> > >> See <https://hudson.apache.org/hudson/job/Shindig/1793/>
> >> > >>
> >> > >>
> >> > >>
> >> > >
> >> >
> >>
> >>
> >>
> >> --
> >> Paul Lindner -- plindner@linkedin.com -- linkedin.com/in/plindner
> >>
> >
> >
>



-- 
Paul Lindner -- plindner@linkedin.com -- linkedin.com/in/plindner

Re: Hudson build is back to normal : Shindig #1793

Posted by Gagandeep singh <ga...@gmail.com>.
I have a doubt. A fellow colleague tells me that we should retain the order
in which the keys are inserted into the query params map, because some ppl
do funny stuff like their own begin and end parameters and mark useful
parameters in between.

I was wondering if we should go with a SortedMap approach for query params
instead. I mean do we really need to bother about query param P1 added
before query param P2 ? Also, tomorrow if some1 refactores some code, the
tests should not fail because the order of insertion has changed.

As a side note, i think Google uses LinkedListMultimap for query params.
Isn't that better than LinkedHashMap<String, List<String>> ?

Thoughts ?



On Mon, Aug 30, 2010 at 1:31 PM, Gagandeep singh <ga...@gmail.com>wrote:

> Nice debugging.
>
>
> On Mon, Aug 30, 2010 at 7:42 AM, Paul Lindner <pl...@linkedin.com>wrote:
>
>> The test output showed that the problem was incorrectly ordered query
>> params.  I validated that the problem occurs only on Java 1.5.  Therefore
>> it
>> had to be a HashMap iteration problem somewhere.  So I poked around
>> UriBuilder and found one remaining use of HashMap and replaced it with
>> LinkedHashMap.
>>
>> On Sun, Aug 29, 2010 at 6:54 PM, Gagandeep singh <gagan.goku@gmail.com
>> >wrote:
>>
>> > Could you also tell us how you found and fixed the problem ?
>> >
>> > On Mon, Aug 30, 2010 at 7:24 AM, Gagandeep singh <gagan.goku@gmail.com
>> > >wrote:
>> >
>> > > Thanks for fixing this Paul.
>> > >
>> > >
>> > > On Mon, Aug 30, 2010 at 2:46 AM, Apache Hudson Server <
>> > > hudson@hudson.apache.org> wrote:
>> > >
>> > >> See <https://hudson.apache.org/hudson/job/Shindig/1793/>
>> > >>
>> > >>
>> > >>
>> > >
>> >
>>
>>
>>
>> --
>> Paul Lindner -- plindner@linkedin.com -- linkedin.com/in/plindner
>>
>
>

Re: Hudson build is back to normal : Shindig #1793

Posted by Gagandeep singh <ga...@gmail.com>.
Nice debugging.

On Mon, Aug 30, 2010 at 7:42 AM, Paul Lindner <pl...@linkedin.com> wrote:

> The test output showed that the problem was incorrectly ordered query
> params.  I validated that the problem occurs only on Java 1.5.  Therefore
> it
> had to be a HashMap iteration problem somewhere.  So I poked around
> UriBuilder and found one remaining use of HashMap and replaced it with
> LinkedHashMap.
>
> On Sun, Aug 29, 2010 at 6:54 PM, Gagandeep singh <gagan.goku@gmail.com
> >wrote:
>
> > Could you also tell us how you found and fixed the problem ?
> >
> > On Mon, Aug 30, 2010 at 7:24 AM, Gagandeep singh <gagan.goku@gmail.com
> > >wrote:
> >
> > > Thanks for fixing this Paul.
> > >
> > >
> > > On Mon, Aug 30, 2010 at 2:46 AM, Apache Hudson Server <
> > > hudson@hudson.apache.org> wrote:
> > >
> > >> See <https://hudson.apache.org/hudson/job/Shindig/1793/>
> > >>
> > >>
> > >>
> > >
> >
>
>
>
> --
> Paul Lindner -- plindner@linkedin.com -- linkedin.com/in/plindner
>

Re: Hudson build is back to normal : Shindig #1793

Posted by Paul Lindner <pl...@linkedin.com>.
The test output showed that the problem was incorrectly ordered query
params.  I validated that the problem occurs only on Java 1.5.  Therefore it
had to be a HashMap iteration problem somewhere.  So I poked around
UriBuilder and found one remaining use of HashMap and replaced it with
LinkedHashMap.

On Sun, Aug 29, 2010 at 6:54 PM, Gagandeep singh <ga...@gmail.com>wrote:

> Could you also tell us how you found and fixed the problem ?
>
> On Mon, Aug 30, 2010 at 7:24 AM, Gagandeep singh <gagan.goku@gmail.com
> >wrote:
>
> > Thanks for fixing this Paul.
> >
> >
> > On Mon, Aug 30, 2010 at 2:46 AM, Apache Hudson Server <
> > hudson@hudson.apache.org> wrote:
> >
> >> See <https://hudson.apache.org/hudson/job/Shindig/1793/>
> >>
> >>
> >>
> >
>



-- 
Paul Lindner -- plindner@linkedin.com -- linkedin.com/in/plindner

Re: Hudson build is back to normal : Shindig #1793

Posted by Gagandeep singh <ga...@gmail.com>.
Could you also tell us how you found and fixed the problem ?

On Mon, Aug 30, 2010 at 7:24 AM, Gagandeep singh <ga...@gmail.com>wrote:

> Thanks for fixing this Paul.
>
>
> On Mon, Aug 30, 2010 at 2:46 AM, Apache Hudson Server <
> hudson@hudson.apache.org> wrote:
>
>> See <https://hudson.apache.org/hudson/job/Shindig/1793/>
>>
>>
>>
>

Re: Hudson build is back to normal : Shindig #1793

Posted by Gagandeep singh <ga...@gmail.com>.
Thanks for fixing this Paul.

On Mon, Aug 30, 2010 at 2:46 AM, Apache Hudson Server <
hudson@hudson.apache.org> wrote:

> See <https://hudson.apache.org/hudson/job/Shindig/1793/>
>
>
>

Hudson build is back to normal : Shindig #1793

Posted by Apache Hudson Server <hu...@hudson.apache.org>.
See <https://hudson.apache.org/hudson/job/Shindig/1793/>