[GitHub] [jackrabbit-oak] rishabhdaim commented on pull request #560: OAK-9757 : increased node name limit for mongo 4.2 version

2022-05-08 Thread GitBox
rishabhdaim commented on PR #560: URL: https://github.com/apache/jackrabbit-oak/pull/560#issuecomment-1120661592 @mreutegg > To me it feels wrong that Utils.isLongPath() is currently responsible for detecting when a node name is too long Yes, I think it should be other way

[GitHub] [jackrabbit-oak] rishabhdaim commented on pull request #560: OAK-9757 : increased node name limit for mongo 4.2 version

2022-05-06 Thread GitBox
rishabhdaim commented on PR #560: URL: https://github.com/apache/jackrabbit-oak/pull/560#issuecomment-1119294326 @reschke That's a good idea to move util function to DocumentStore and that would help us avoiding in calling metadata multiple times. Let me make changes and raise the PR