Sorry - there were a bunch of issues on master:

+ looks like there was a bit of a merge problem at some point between tp32
and master. I added some new types to gryo in 3.2.x but the new
registration ids I used were already in use in 3.3.x. Luckily, we have good
error messaging around gryo registration errors so it was easy to figure
out where the conflict was.
+ noticed a problem with my new ClassSerializer for gryo that wasn't
working right on master.
+ we were missing a gryo registration for GreedyMatchAlgorithm - not sure
why that wasn't a problem before

I've pushed the fixes and master should be good again.  Travis is happy:

https://travis-ci.org/apache/tinkerpop/builds/185586900

Thanks for pointing that we had problems there. I'm not sure why it wasn't
noticed earlier.



On Tue, Dec 20, 2016 at 2:27 PM, Stephen Mallette <[email protected]>
wrote:

> hmmm - travis has a similar issue. looking at it now.
>
> On Tue, Dec 20, 2016 at 2:24 PM, karthik tunga <[email protected]>
> wrote:
>
>> Hi,
>>
>> I am new to Tinkerpop. I tried to build the code from GitHub and there are
>> unit test failures. Tests fail with the below error. Gremlin Core fails to
>> build.
>> Can anyone help ?
>>
>> GryoMapperTest.<init>:78 » IllegalState There are duplicate kryo
>> identifiers i...
>>
>> GryoPoolTest.shouldConfigPoolOnConstructionWithMultipleCusto
>> mIoRegistries:107
>> » IllegalState
>>   GryoPoolTest.shouldDoWithReaderWriterMethods:44 » IllegalState There
>> are
>> dupli...
>>   GryoPoolTest.shouldConfigPoolOnConstructionWithDefaults:58 »
>> IllegalState
>> Ther...
>>   GryoPoolTest.shouldConfigPoolOnConstructionWithoutCustomIoRe
>> gistryAndFail
>> »  U...
>>
>> GryoPoolTest.shouldConfigPoolOnConstructionWithCustomIoRegis
>> tryInstance:98
>> » IllegalState
>>
>> GryoPoolTest.shouldConfigPoolOnConstructionWithCustomIoRegis
>> tryConstructor:90
>> » IllegalState
>>
>> GryoPoolTest.shouldConfigPoolOnConstructionWithPoolSizeOneAn
>> dNoIoRegistry:65
>> » IllegalState
>>   GryoReaderWriterTest.shouldBeAbleToReUseBuilderInDifferentThreads:42 »
>> IllegalState
>>
>> Thanks.
>>
>> Cheers,
>> Karthik
>>
>
>

Reply via email to