rok commented on issue #49153: URL: https://github.com/apache/arrow/issues/49153#issuecomment-3856844335
@AliRana30, while these symbols appear to be deprecated your message is not completely correct. I only checked point 3. > 3. **`ServerAuthHandler::IsValid()` old signature** > > * Deprecated in **v13.0.0** (11+ years ago) > * Located in the Flight module `ServerAuthHandler::IsValid()` was deprecated [here](https://github.com/apache/arrow/pull/35378/changes#diff-b58785bae90c3fb7919083176bc44b2f4ce6e703e31ae28d8333c2b2c98413b8R97) on May 2nd 2023. This was part of [13.0.0](https://arrow.apache.org/release/13.0.0.html) release on 23rd August 2023. So while you have found a deprecated symbol you posted potentially misleading (and in this case unnecessary) information that I assume was AI generated. This is what we would prefer to avoid by contributors manually verifying information and not implicitly asking maintainers to do so. Proposal to start with a c++ PR is ok, but please do read and understand what you are posting, to quote from [here](https://arrow.apache.org/docs/dev/developers/overview.html#ai-generated-code): > When a PR appears to have been generated without much engagement from the submitter, reviewers with access to AI tools could more efficiently generate the code directly, and since the submitter is not likely to learn from the review process, their time is more productively spent researching and reporting on the issue. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
