Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TestThriftToPig for Elephantbird piggybank getting failed #468

Open
SonaliShrivastava opened this issue Jun 15, 2016 · 5 comments
Open

TestThriftToPig for Elephantbird piggybank getting failed #468

SonaliShrivastava opened this issue Jun 15, 2016 · 5 comments

Comments

@SonaliShrivastava
Copy link

Hi All,

I am trying to port Elephant Bird Stable Release "2.2.3" on RHEL 7.2 ppc64le using open jdk 1.8
Got success in running ant, ant compile, ant resolve, ant jar and for other targets too except got
failure for ant test.

It gives me Test Failure in below line:
[junit] Running com.twitter.elephantbird.pig.piggybank.TestThriftToPig
[junit] Tests run: 16, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.081 sec
[junit] TEST com.twitter.elephantbird.pig.piggybank.TestThriftToPig FAILED

By debugging failures,
I see, it is failing on line no. 160 i.e.
assertEquals("{(1,0,34,27000,16777216,6000000000,3.141592653589793,JSON THIS! "^A," + ooe.zomg_unicode +
",0,base64,{(1),(2),(3)},{(1),(2),(3)},{(1),(2),(3)}),(1,0,35,27000,16777216,6000000000,3.141592653589793,JSON THIS! "^A," +
ooe.zomg_unicode + ",0,base64,{(1),(2),(3)},{(1),(2),(3)},{(1),(2),(3)})}-{({}),({(and a one),(and a two)}),({(then a one, two),(three!),(FOUR!!)})}-{zero={}, three={}, two={(1,Wait.),(2,What?)}}",
(toTuple(type, hm).toDelimitedString("-")));

For debugging purpose, i have added logs and could get below clarity for string comparison failing as a result they are not equal and so assert error, following are details log i printed:
FAILED
expected:<...,{(1),(2),(3)})}-{({[}),({(and a one),(and a two)}),({(then a one, two),(three!),(FOUR!!)})}-{zero={}, three={}, two={(1,Wait.),(2,What?)]}}>
but was:<...,{(1),(2),(3)})}-{({[(and a one),(and a two)}),({}),({(then a one, two),(three!),(FOUR!!)})}-{zero={}, two={(1,Wait.),(2,What?)}, three={]}}>
junit.framework.AssertionFailedError.

If I comment above line, this happens similar to line below too:
assertEquals("(bob,jenkins),42,foo@bar.com,{(415-555-5555,HOME)}", toTuple(type, person).toDelimitedString(","));

as well as also for line i.e.
assertTrue( // the order of elements in map could vary because of HashMap
tupleString.equals("(bob,jenkins)-{MOBILE=650-555-5555, WORK=415-555-5555, HOME=408-555-5555}") ||
tupleString.equals("(bob,jenkins)-{MOBILE=650-555-5555, HOME=408-555-5555, WORK=415-555-5555}"));

When i comment all 3 lines above, test passes for all other modules of it and also for other projects of elephant-bird.

It seems this is coding issue.
@Team: Can you please let me know for reason it is failing beside other all test are passing except failing for com.twitter.elephantbird.pig.piggybank.TestThriftToPig.

Thanks & Regards,
Sonali Shrivastava

@jagadale
Copy link

Dear Team, Any comment on this issue?

@SonaliShrivastava
Copy link
Author

Hello Team,

I have build elephant-bird for "RHEL7.1 x86-64" too and could find same above error i.e.
Running com.twitter.elephantbird.pig.piggybank.TestThriftToPig
[junit] Tests run: 16, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.315 sec
[junit] TEST com.twitter.elephantbird.pig.piggybank.TestThriftToPig FAILED

It seems to be coding error as explained above.
Can you please let us know for the reason it is failing? Waiting for your reply.

Thanks & Regards,
Sonali Shrivastava

@rubanm
Copy link
Contributor

rubanm commented Jun 22, 2016

@SonaliShrivastava Sorry, elephant-bird-pig is not actively being developed. And 2.2.3 is a pretty old version (May 2012). We are currently on 4.14. Any reason you're using that version?

On the latest version, you should be able to run the release.sh script to run all the tests. See:
https://github.com/twitter/elephant-bird/blob/master/release.sh#L201
https://github.com/twitter/elephant-bird/wiki/Release-Howto#running-with-vagrant

@SonaliShrivastava
Copy link
Author

HI Rubanm,

Yes 2.2.3 is stable release and so have gone for it. Thanks for reply.
As a reason for still development going on it and hence the issue for ppc64le and x86-64 too.

Thanks & Regards,
Sonali Shrivastava

@isnotinvain
Copy link
Contributor

@SonaliShrivastava I'm not sure where 2.2.3 is coming from, but the current release is 4.14
Do we have some old documentation claiming 2.2.3 is the stable release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants