You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "niyue (via GitHub)" <gi...@apache.org> on 2023/09/16 09:34:00 UTC
[GitHub] [arrow] niyue opened a new issue, #37751: [C++] [Gandiva] Exported functions are registered multiple times in gandiva
niyue opened a new issue, #37751:
URL: https://github.com/apache/arrow/issues/37751
### Describe the bug, including details regarding any error messages, version, and platform.
The exported functions defined in gandiva are registered multiple times unnecessarily.
Currently, in gandiva, exported functions are registered to the function registry in `exported_funcs.h` by using a macro `REGISTER_EXPORTED_FUNCS`, which internally defines a static variable to call the registration. Since this header file (`exported_funcs.h`) is included in multiple times in different files (7 different files more specifically), this makes the registration happens 7 times for the same set of functions, which is not necessary. It makes the initialization slower, additionally, it makes the registration to contain duplicated functions (although it still works so far).
### Component(s)
C++ - Gandiva
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [C++] [Gandiva] Exported functions are registered multiple times in gandiva [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou closed issue #37751: [C++] [Gandiva] Exported functions are registered multiple times in gandiva
URL: https://github.com/apache/arrow/issues/37751
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [C++] [Gandiva] Exported functions are registered multiple times in gandiva [arrow]
Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou commented on issue #37751:
URL: https://github.com/apache/arrow/issues/37751#issuecomment-1764023338
Issue resolved by pull request 37752
https://github.com/apache/arrow/pull/37752
--
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