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

failing test TestS3PartitionedFileListing #2

Open
jorkzijlstra opened this issue May 17, 2017 · 5 comments
Open

failing test TestS3PartitionedFileListing #2

jorkzijlstra opened this issue May 17, 2017 · 5 comments

Comments

@jorkzijlstra
Copy link

jorkzijlstra commented May 17, 2017

Assert.assertEquals("File sets should match", expectedFiles, actualFiles);

expectedFiles have the same order as the actualFiles

java.lang.AssertionError: File sets should match expected:

<[dateint=20161115/hour=13/620acc75-14f1-44a8-8cdf-c7853d816c31.parquet, 
dateint=20161115/hour=14/059f4eab-cc21-4ab5-a4e8-ceedbc9e52cf.parquet, 
dateint=20161116/hour=13/8e87ab9d-f40f-4a5f-b61e-8f613b97fd6a.parquet, 
dateint=20161116/hour=14/af18c4db-5ade-40bc-add1-c9984c22b445.parquet]> 

but was:
<[dateint=20161116/hour=14/af18c4db-5ade-40bc-add1-c9984c22b445.parquet, 
dateint=20161116/hour=13/8e87ab9d-f40f-4a5f-b61e-8f613b97fd6a.parquet, 
dateint=20161115/hour=14/059f4eab-cc21-4ab5-a4e8-ceedbc9e52cf.parquet, 
dateint=20161115/hour=13/620acc75-14f1-44a8-8cdf-c7853d816c31.parquet]>
@jorkzijlstra
Copy link
Author

TestS3PartitionedFileListing. testTaskOutputListingWithHiddenFiles also have the same issue

@ftago
Copy link

ftago commented Jul 15, 2017

I see the same failures .
com.netflix.bdp.s3.TestS3PartitionedFileListing > testTaskOutputListingWithHiddenFiles
com.netflix.bdp.s3.TestS3PartitionedFileListing > testTaskOutputListing

What is the update on this ?

@jzijlstra
Copy link

@ftago I haven't heard anything from @rdblue since I created the issue

@yiftachdy
Copy link

@ftago @jorkzijlstra @jorkzijlstra Need to change the expected and actual lists to sets - so order wont matter

@rdblue
Copy link
Owner

rdblue commented Aug 8, 2018

I think there's an easy fix to use sets. If you open a PR, I can merge the fix.

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

5 participants