westonpace commented on pull request #10205: URL: https://github.com/apache/arrow/pull/10205#issuecomment-840311677
Ok, using @bkietz 's suggestions I was able to collapse the two classes back into one. I still have some cleanup to do so this is not ready for review yet. The result is more compact, although I'm not necessarily convinced it is any more maintainable but :shrug: . -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
