Skip to content

Conversation

@wesm
Copy link
Member

@wesm wesm commented Dec 7, 2017

These were showing up in our Doxygen docs and may mislead users reading the public API into thinking these classes do something that they do not (they don't read general JSON)

…xport symbols

Change-Id: Ib849ed13a843d602f3dcb43d51197a760e94bca0
Copy link
Member

@xhochy xhochy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM

@wesm wesm closed this in 64af87b Dec 7, 2017
@wesm wesm deleted the ARROW-1884 branch December 7, 2017 20:54
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