Chris Olivier

Results 9 comments of Chris Olivier

+1 for using these macros. For one, it makes the project portable to different computers. For another, it's basically required for CLion remote mode (pretty common these days where development...

I also show leaks with asan only for _enum (every enum_) in make_function_record()

This PR has conflicts that would need to be resolved before review.

@smallcampus If this is still desired, can please resolve any conflicts?

Is this still relevant? Does Athena still have problems with non-lowercase S3 paths?

@danosipov would you be able to add some tests?

I also get this problem when loading tensorflow via pybind if I have *the same version* of protobuf also in my c++ code