You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/16 15:57:38 UTC

[GitHub] [arrow] js8544 commented on pull request #14657: ARROW-18343: [C++] AllocateBitmap() with out parameter is declared but not defined

js8544 commented on PR #14657:
URL: https://github.com/apache/arrow/pull/14657#issuecomment-1317244454

   > Can you find out since when that has been the case? If there have been at least two releases without, perhaps we can safely remove it :-)
   
   The definition is removed in [this PR](https://github.com/apache/arrow/pull/7335/files#diff-590dc662378ffd51c79ee710b3058838e95a9eb16877435b131018f0becb9faaL307) but the api is kept. It appeared to be a mistake to remove the definition back then because this api is not marked deprecated in [this PR](https://github.com/apache/arrow/pull/6863/files#diff-2edce6fb9f08ce495de42b07497d9d43d57b12051ef5aa997ad6056eb207b70dR514) (or perhaps it should be marked deprecated but was overlooked). Either way since it has been like this for over 2 years, perhaps it's best to remove this api?


-- 
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.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org