rovboyko commented on PR #21406:
URL: https://github.com/apache/flink/pull/21406#issuecomment-2014338903

   @Myasuka after implementation of 
[FLINK-30535](https://issues.apache.org/jira/browse/FLINK-30535) and 
[FLINK-33881](https://issues.apache.org/jira/browse/FLINK-33881) I changed the 
current commit to optimize only TtlMapState.
   
   After the optimization benchamrk results seem not so bad (+17% performance):
   
   Benchmark | backendType | expiredOption | stateVisibility | updateType | 
Non-optimized | Optimized | Units
   -- | -- | -- | -- | -- | -- | -- | --
   TtlMapStateBenchmark.mapAdd | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnCreateAndWrite | 361.671 | 521.839 | ops/ms
   TtlMapStateBenchmark.mapAdd | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnReadAndWrite | 366.513 | 523.892 | ops/ms
   TtlMapStateBenchmark.mapAdd | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 364.311 | 461.037 | ops/ms
   TtlMapStateBenchmark.mapAdd | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 362.902 | 510.513 | ops/ms
   TtlMapStateBenchmark.mapAdd | HEAP | NeverExpired | NeverReturnExpired | 
OnCreateAndWrite | 361.08 | 520.319 | ops/ms
   TtlMapStateBenchmark.mapAdd | HEAP | NeverExpired | NeverReturnExpired | 
OnReadAndWrite | 364.026 | 524.573 | ops/ms
   TtlMapStateBenchmark.mapAdd | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 362.199 | 515.707 | ops/ms
   TtlMapStateBenchmark.mapAdd | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 357.642 | 433.35 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnCreateAndWrite | 304.937 | 386.748 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnReadAndWrite | 260.486 | 321.793 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 324.409 | 397.574 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 265.357 | 353.769 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | NeverExpired | NeverReturnExpired | 
OnCreateAndWrite | 328.767 | 405.722 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | NeverExpired | NeverReturnExpired | 
OnReadAndWrite | 255.596 | 338.208 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 317.633 | 400.027 | ops/ms
   TtlMapStateBenchmark.mapGet | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 254.176 | 343.937 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnCreateAndWrite | 316.203 | 412.834 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnReadAndWrite | 327.426 | 414.538 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 323.163 | 406.269 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 334.041 | 420.389 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | NeverExpired | NeverReturnExpired | 
OnCreateAndWrite | 346.116 | 406.019 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | NeverExpired | NeverReturnExpired | 
OnReadAndWrite | 320.814 | 405.261 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 340.874 | 409.718 | ops/ms
   TtlMapStateBenchmark.mapIsEmpty | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 333.791 | 408.396 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnCreateAndWrite | 2760.188 | 3413.68 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnReadAndWrite | 2322.375 | 2419.871 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 2694.602 | 3372.643 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 2275.37 | 2156.808 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | NeverExpired | NeverReturnExpired 
| OnCreateAndWrite | 2723.703 | 3369.097 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | NeverExpired | NeverReturnExpired 
| OnReadAndWrite | 2324.465 | 2453.601 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 2694.324 | 3364.168 | ops/ms
   TtlMapStateBenchmark.mapIterator | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 2339.08 | 2377.672 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnCreateAndWrite | 270.326 | 275.442 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnReadAndWrite | 269.82 | 282.63 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 271.646 | 280.797 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 270.449 | 257.802 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | NeverExpired | NeverReturnExpired | 
OnCreateAndWrite | 271.497 | 274.694 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | NeverExpired | NeverReturnExpired | 
OnReadAndWrite | 270.721 | 252.761 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 292.303 | 268.975 | ops/ms
   TtlMapStateBenchmark.mapPutAll | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 271.283 | 277.794 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnCreateAndWrite | 278.52 | 345.521 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | Expire3PercentPerIteration | 
NeverReturnExpired | OnReadAndWrite | 272.15 | 328.734 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 274.306 | 358.366 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | Expire3PercentPerIteration | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 272.916 | 347.087 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | NeverExpired | NeverReturnExpired | 
OnCreateAndWrite | 264.072 | 346.595 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | NeverExpired | NeverReturnExpired | 
OnReadAndWrite | 273.377 | 341.417 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnCreateAndWrite | 275.702 | 352.406 | ops/ms
   TtlMapStateBenchmark.mapUpdate | HEAP | NeverExpired | 
ReturnExpiredIfNotCleanedUp | OnReadAndWrite | 275.58 | 357.394 | ops/ms
     |   |   |   | total: | 32362.908 | 38118.387 |  
   
   
   </body>
   
   </html>
   


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to