You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We would like to hide the internal modules from the public API. After discussion, there are three options:
Publicly export the internal modules under the Database.LSMTree.Internal namespace.
Migrate the internal modules to a private sub-library.
Migrate all modules to a private sub-library,
then create modules in the public library that re-export the public modules.
Our consensus is that we should not make any of these changes now, since all of these affect too many files, but that we should settle on which of the three options we should use before we cut our first release.
Overall, we felt positive about options (2) or (3), but were slightly divided between the two options. Option (3) is likely easier for developers and maintainers, but there was a fear that it might have a slightly higher performance impact. Hence, if we choose option (2) or (3) we should benchmark the change to see if there is any performance impact from the potential loss of inlining that might result from this rewrite.
The text was updated successfully, but these errors were encountered:
We would like to hide the internal modules from the public API. After discussion, there are three options:
Database.LSMTree.Internal
namespace.then create modules in the public library that re-export the public modules.
Our consensus is that we should not make any of these changes now, since all of these affect too many files, but that we should settle on which of the three options we should use before we cut our first release.
Overall, we felt positive about options (2) or (3), but were slightly divided between the two options. Option (3) is likely easier for developers and maintainers, but there was a fear that it might have a slightly higher performance impact. Hence, if we choose option (2) or (3) we should benchmark the change to see if there is any performance impact from the potential loss of inlining that might result from this rewrite.
The text was updated successfully, but these errors were encountered: