renato2099 commented on PR #18672: URL: https://github.com/apache/datafusion/pull/18672#issuecomment-3568419719
Hi @timsaucer , Cool work! I have a question about the overall approach, but please excuse me if this a naive/silly question , I am still trying to get familiar with the approach and the code base :) Right now we are relying on memory address of a static to determine if we are in a foreign/local context, and although that raw memory check makes it now, I am not sure if having some sort of global registry would help future proofing the whole improvement. The global registry could still hold the raw memory pointer but also some UUID generated when a new library gets registered. Do you think that would help? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
