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

[C++] Move JSON testing code for integration tests to libarrow_testing #25347

Closed
asfimport opened this issue Jun 27, 2020 · 1 comment
Closed

Comments

@asfimport
Copy link

This code contributes over 700KB to release builds and is never used

-rw------- 1 wesm wesm  34104 Jun 27 11:14 dictionary.cc.o
-rw------- 1 wesm wesm 199592 Jun 27 11:14 feather.cc.o
-rw------- 1 wesm wesm  63448 Jun 27 11:14 json_integration.cc.o
-rw------- 1 wesm wesm 727336 Jun 27 11:14 json_internal.cc.o
-rw------- 1 wesm wesm 828056 Jun 27 11:14 json_simple.cc.o
-rw------- 1 wesm wesm 185344 Jun 27 11:14 message.cc.o
-rw------- 1 wesm wesm 223592 Jun 27 11:14 metadata_internal.cc.o
-rw------- 1 wesm wesm   3416 Jun 27 11:14 options.cc.o
-rw------- 1 wesm wesm 557960 Jun 27 11:14 reader.cc.o
-rw------- 1 wesm wesm 285744 Jun 27 11:14 writer.cc.o

Reporter: Wes McKinney / @wesm
Assignee: Wes McKinney / @wesm

PRs and other links:

Note: This issue was originally created as ARROW-9251. Please see the migration documentation for further details.

@asfimport
Copy link
Author

Wes McKinney / @wesm:
Issue resolved by pull request 7557
#7557

@asfimport asfimport added this to the 1.0.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants