ankitsultana commented on PR #12538:
URL: https://github.com/apache/pinot/pull/12538#issuecomment-2000468919
I ran some tests and I see that old-gen is worse with this change vs not
setting any hash-function at all.
```
num #instances #bytes class name (module)
-------------------------------------------------------
1: 123313867 5919065616
java.util.concurrent.ConcurrentHashMap$Node ([email protected])
2: 123395855 4961387056 [B ([email protected])
3: 123212269 4928490760
org.apache.pinot.segment.local.upsert.ConcurrentMapPartitionUpsertMetadataManager$RecordLocation
4: 123212269 3942792608 org.apache.pinot.spi.utils.ByteArray
5: 10901 2153606592 [I ([email protected])
6: 907 2149323976
[Ljava.util.concurrent.ConcurrentHashMap$Node; ([email protected])
7: 14747 24051696 [C ([email protected])
8: 2954 18350504 [J ([email protected])
...
```
```
num #instances #bytes class name (module)
-------------------------------------------------------
1: 123392066 7918671528 [B ([email protected])
2: 123310544 5918906112
java.util.concurrent.ConcurrentHashMap$Node ([email protected])
3: 123209015 4928360600
org.apache.pinot.segment.local.upsert.ConcurrentMapPartitionUpsertMetadataManager$RecordLocation
4: 123245008 3956561736 [Ljava.lang.Object; ([email protected])
5: 123378686 3948117952 java.lang.String ([email protected])
6: 123209015 2957016360
org.apache.pinot.spi.data.readers.PrimaryKey
7: 11013 2153608640 [I ([email protected])
8: 921 2149325080
[Ljava.util.concurrent.ConcurrentHashMap$Node; ([email protected])
...
```
--
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]
