Skip to content

Conversation

@kiszk
Copy link
Member

@kiszk kiszk commented Jul 26, 2020

Add golden files for big and little endians for apache/arrow#7507 based on @wesm 's suggestion.

@kiszk
Copy link
Member Author

kiszk commented Jul 26, 2020

cc @wesm

@wesm
Copy link
Member

wesm commented Jul 27, 2020

Nit: do we want maybe to put these files in directories like

  • integration/1.0.0-littleendian
  • integration/1.0.0-bigendian

? Is there a script somewhere for generating these files? I think I have asked this in the past and I'm still not sure the answer

@kiszk
Copy link
Member Author

kiszk commented Jul 28, 2020

Should I put a set of scripts into integration/1.0.0-littleendian and integration/1.0.0-bigendian? (Same set for both endian).
Or, should I put the set here as a comment of this PR?

@wesm
Copy link
Member

wesm commented Jul 28, 2020

Probably the script should be a part of apache/arrow and be used to generate a set of golden files at a point in time for a particular build, maybe it could be a part of the archery tool or simply a bash script would be OK too

@wesm wesm merged commit 0e56bdd into apache:master Jul 28, 2020
@kiszk
Copy link
Member Author

kiszk commented Jul 29, 2020

I see. I will add a bash script to apache/arrow#7507

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

Successfully merging this pull request may close these issues.

2 participants