{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":504360264,"defaultBranch":"master","name":"incubator-uniffle","ownerLogin":"apache","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2022-06-17T01:47:53.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/47359?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1716873968.0","currentOid":""},"activityList":{"items":[{"before":"56f3de0575c2458935adfc2f3936ab11ad39cbeb","after":"a6a715fb2bf6660253299608b18fb1794dddb06e","ref":"refs/heads/branch-0.9","pushedAt":"2024-06-04T12:49:39.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"EnricoMi","name":"Enrico Minack","path":"/EnricoMi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/44700269?s=80&v=4"},"commit":{"message":"[#1764] fix(client): Fix timeout time unit for unregister requests (#1766)\n\n### What changes were proposed in this pull request?\r\nMethod `ThreadUtils.executeTasks` expects milliseconds, but seconds are given.\r\n\r\n### Why are the changes needed?\r\nRequests are interrupted before the configured timeout passes.\r\n\r\nFix: #1764\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\nConfigured timeout is respected.\r\n\r\n### How was this patch tested?\r\nManually tested.","shortMessageHtmlLink":"[#1764] fix(client): Fix timeout time unit for unregister requests (#…"}},{"before":"eb164a8fa79cf70dcdbb13774686663cf11bc8af","after":"41d78aafac933e7c7ffa5dddd440b7465a3bb63e","ref":"refs/heads/master","pushedAt":"2024-06-04T08:53:17.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1764] fix(client): Fix timeout time unit for unregister requests (#1766)\n\n### What changes were proposed in this pull request?\r\nMethod `ThreadUtils.executeTasks` expects milliseconds, but seconds are given.\r\n\r\n### Why are the changes needed?\r\nRequests are interrupted before the configured timeout passes.\r\n\r\nFix: #1764\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\nConfigured timeout is respected.\r\n\r\n### How was this patch tested?\r\nManually tested.","shortMessageHtmlLink":"[#1764] fix(client): Fix timeout time unit for unregister requests (#…"}},{"before":"27d6fca88354a26a1b51284c82b4fde639c60e8b","after":"eb164a8fa79cf70dcdbb13774686663cf11bc8af","ref":"refs/heads/master","pushedAt":"2024-06-03T07:55:25.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1755][FOLLOWUP] fix(server): Incorrect request info to output too much logs (#1760)\n\n### What changes were proposed in this pull request?\r\n\r\nFix incorrect request to make output too much logs\r\n\r\n### Why are the changes needed?\r\n\r\nFollowup for #1755 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nNeedn't","shortMessageHtmlLink":"[#1755][FOLLOWUP] fix(server): Incorrect request info to output too m…"}},{"before":"5e49017672820533bf5a59d1e8bc0fa5bae95978","after":"27d6fca88354a26a1b51284c82b4fde639c60e8b","ref":"refs/heads/master","pushedAt":"2024-06-03T06:50:03.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1699][FOLLOWUP] fix(client): Add commons-collections4 dependencies in shaded clients (#1742)\n\n### What changes were proposed in this pull request?\r\n\r\nAdd `commons-collections4` dependencies in shaded clients.\r\n\r\n### Why are the changes needed?\r\n\r\nAfter https://github.com/apache/incubator-uniffle/pull/1700, in some old versions of Spark, the `commons-collections4` dependency could be missing. Add the potential missing dependency to improve robustness.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting UTs. Tested in our env.","shortMessageHtmlLink":"[#1699][FOLLOWUP] fix(client): Add commons-collections4 dependencies …"}},{"before":"414d5cf51848b67b0a0e0fc23df19d67a5741f1b","after":"5e49017672820533bf5a59d1e8bc0fa5bae95978","ref":"refs/heads/master","pushedAt":"2024-06-03T06:49:32.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[MINOR] fix(dashboard): Display the registration time in 24-hour format (#1752)\n\n### What changes were proposed in this pull request?\r\n\r\nDisplay the registration time in 24-hour format.\r\n\r\n### Why are the changes needed?\r\n\r\nThe current displayed registration time follows the system time of the machine. If the system is set to a 12-hour format, then the page will display the time in a 12-hour format. If the system is set to a 24-hour format, then the page will display the time in a 24-hour format. It's not clear in the UI whether it's AM or PM when the Linux machine uses the 12-hour format.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting UTs.","shortMessageHtmlLink":"[MINOR] fix(dashboard): Display the registration time in 24-hour form…"}},{"before":"fd64d9dcc2e4eb26aa4da1a6fd4a5f07db95845f","after":"414d5cf51848b67b0a0e0fc23df19d67a5741f1b","ref":"refs/heads/master","pushedAt":"2024-06-03T03:30:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1711] feat(server): Make the health checker execution timeout reconfigurable (#1754)\n\n### What changes were proposed in this pull request?\r\n\r\n1. Make the health checker execution timeout reconfigurable\r\n2. Introduce the `rssConf.getReconfigurableConf` to use the reconfigurable conf.\r\n\r\n### Why are the changes needed?\r\n\r\nFor #1711\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting tests.","shortMessageHtmlLink":"[#1711] feat(server): Make the health checker execution timeout recon…"}},{"before":"d182a039685e4c50d5b500b9ac7aee52dcec623c","after":"fd64d9dcc2e4eb26aa4da1a6fd4a5f07db95845f","ref":"refs/heads/master","pushedAt":"2024-06-03T02:14:54.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1757] feat(server): Add block number check on getting shuffle result (#1758)\n\n### What changes were proposed in this pull request?\r\n\r\nAdd block number check on getting shuffle result \r\n\r\n### Why are the changes needed?\r\n\r\nData validation, ensure data stable and correct\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting tests.","shortMessageHtmlLink":"[#1757] feat(server): Add block number check on getting shuffle result ("}},{"before":"8d49063115ee49b18c366e8cf4c4b65de9782b45","after":"d182a039685e4c50d5b500b9ac7aee52dcec623c","ref":"refs/heads/master","pushedAt":"2024-05-30T06:51:23.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1755] fix(spark): Avoid task failure of inconsistent record number (#1756)\n\n### What changes were proposed in this pull request?\r\n\r\n1. When the spill ratio is `1.0` , the process of calculating target spill size will be ignored to avoid potential race condition that the `usedBytes` and `inSendBytes` are not thread safe. This could guarantee that the all data is flushed to the shuffle server at the end of task.\r\n2. Adding the `bufferManager's` buffer remaining check\r\n\r\n### Why are the changes needed?\r\n\r\nDue to the #1670 , the partial data held by the bufferManager will not be flushed to shuffle servers in some corner cases, \r\nthis will make task fail fast rather than silently data loss that should thanks the #1558 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting tests.","shortMessageHtmlLink":"[#1755] fix(spark): Avoid task failure of inconsistent record number (#…"}},{"before":"a3a49f0e1bfc8a15c408b4f65774b6a5e7a991fc","after":"8d49063115ee49b18c366e8cf4c4b65de9782b45","ref":"refs/heads/master","pushedAt":"2024-05-30T06:50:07.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1743] fix: Add exception handling for thread pools (#1744)\n\n### What changes were proposed in this pull request?\r\n\r\nAdd exception handling for thread pools.\r\n\r\n### Why are the changes needed?\r\n\r\nFix: https://github.com/apache/incubator-uniffle/issues/1743.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting UTs.","shortMessageHtmlLink":"[#1743] fix: Add exception handling for thread pools (#1744)"}},{"before":"417674d779cc2654f7f0adca2c0a53a0da7f5b4d","after":"a3a49f0e1bfc8a15c408b4f65774b6a5e7a991fc","ref":"refs/heads/master","pushedAt":"2024-05-29T08:01:49.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1711][FOLLOWUP] feat(coordinator): refactor the reconfigurable conf (#1741)\n\n### What changes were proposed in this pull request?\r\n\r\nRefactor the reconfigurable conf for coordinator side.\r\n\r\n### Why are the changes needed?\r\n\r\nFollow up: #1711 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnit tests","shortMessageHtmlLink":"[#1711][FOLLOWUP] feat(coordinator): refactor the reconfigurable conf ("}},{"before":"59e509089baa3eb84d11b39322b5f2f4b478f97f","after":"417674d779cc2654f7f0adca2c0a53a0da7f5b4d","ref":"refs/heads/master","pushedAt":"2024-05-28T05:57:51.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1675][FOLLOWUP] fix(test): Explicitly close resources to avoid unexcepted behaviors (#1740)\n\n### What changes were proposed in this pull request?\r\n\r\nFix various flaky tests:\r\n1. After using `ApplicationManager`, release resources at once, which will let `ScheduledExecutorService` work background to cause unexpected behaviors.\r\n2. Release `DynamicClientConfService` after using it.\r\n3. Release `SimpleClusterManager` everytime after using it. Since the execution order of JUnit test methods is random, unless we specifically set the execution order. So it is better not to initialize the `SimpleClusterManager` object globally.\r\n4. Wait for a while to ensure the port is released in `UniffleJavaProcess`.\r\n5. Always call `stopServer` after `new CoordinatorServer()` to shut down multiple `ExecutorService` from running in background, which could cause unexpected behaviors.\r\n\r\n\r\n### Why are the changes needed?\r\n\r\nFor https://github.com/apache/incubator-uniffle/issues/1675.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnnecessary.","shortMessageHtmlLink":"[#1675][FOLLOWUP] fix(test): Explicitly close resources to avoid unex…"}},{"before":"ae9f8eb6fc0c20201b235bd43f3c334533dfa55f","after":"56f3de0575c2458935adfc2f3936ab11ad39cbeb","ref":"refs/heads/branch-0.9","pushedAt":"2024-05-28T05:25:55.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"EnricoMi","name":"Enrico Minack","path":"/EnricoMi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/44700269?s=80&v=4"},"commit":{"message":"Bump project version to 0.9.0","shortMessageHtmlLink":"Bump project version to 0.9.0"}},{"before":"eaa7af4570eafc42d53cba3c70c0a736edb8082f","after":"ae9f8eb6fc0c20201b235bd43f3c334533dfa55f","ref":"refs/heads/branch-0.9","pushedAt":"2024-05-28T05:14:10.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"EnricoMi","name":"Enrico Minack","path":"/EnricoMi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/44700269?s=80&v=4"},"commit":{"message":"[#1149] fix: GC logs in JDK 11 do not include date and time stamps. (#1240)\n\n### What changes were proposed in this pull request?\r\n\r\nFix GC logs in JDK 11 do not include date and time stamps\r\n\r\n### Why are the changes needed?\r\n\r\nFix: #1149 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nManually tested","shortMessageHtmlLink":"[#1149] fix: GC logs in JDK 11 do not include date and time stamps. (#…"}},{"before":"59b899eb19e3dbca32b6a81251ec17fc21671a52","after":"59e509089baa3eb84d11b39322b5f2f4b478f97f","ref":"refs/heads/master","pushedAt":"2024-05-28T05:13:17.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"EnricoMi","name":"Enrico Minack","path":"/EnricoMi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/44700269?s=80&v=4"},"commit":{"message":"[#1149] fix: GC logs in JDK 11 do not include date and time stamps. (#1240)\n\n### What changes were proposed in this pull request?\r\n\r\nFix GC logs in JDK 11 do not include date and time stamps\r\n\r\n### Why are the changes needed?\r\n\r\nFix: #1149 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nManually tested","shortMessageHtmlLink":"[#1149] fix: GC logs in JDK 11 do not include date and time stamps. (#…"}},{"before":"37d64a437568a1a7030939922fd7b5d883f2959a","after":"59b899eb19e3dbca32b6a81251ec17fc21671a52","ref":"refs/heads/master","pushedAt":"2024-05-28T03:19:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1711][FOLLOWUP] fix(server): Avoid outputing too much incorrect non-update logs (#1737)\n\n### What changes were proposed in this pull request?\r\n\r\nAvoid outputing too much incorrect non-update logs\r\n\r\n### Why are the changes needed?\r\n\r\nfollow up #1711\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nYes","shortMessageHtmlLink":"[#1711][FOLLOWUP] fix(server): Avoid outputing too much incorrect non…"}},{"before":"a0e88da59f9c7c30d43120c7fa88ec1d0736a616","after":"37d64a437568a1a7030939922fd7b5d883f2959a","ref":"refs/heads/master","pushedAt":"2024-05-28T03:11:33.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1698][FOLLOWUP] fix(test): Adjust jvm opts to increase stability of tests (#1739)\n\n### What changes were proposed in this pull request?\r\n\r\nWe adjust the memory arguments refer to Spark's [pom.xml](https://github.com/apache/spark/blob/master/pom.xml):\r\n> -ea -Xmx4g -Xss4m -XX:MaxMetaspaceSize=2g -XX:ReservedCodeCacheSize=${CodeCacheSize} ${extraJavaTestArgs}\r\n\r\nand [make-distribution.sh](https://github.com/apache/spark/blob/master/dev/make-distribution.sh):\r\n> export MAVEN_OPTS=\"${MAVEN_OPTS:--Xss128m -Xmx4g -XX:ReservedCodeCacheSize=128m}\"\r\n\r\nThis is more reasonable.\r\n\r\n### Why are the changes needed?\r\n\r\nFix: https://github.com/apache/incubator-uniffle/issues/1698.\r\nAfter https://github.com/apache/incubator-uniffle/pull/1726, I found that the issue may still exist.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnnecessary.","shortMessageHtmlLink":"[#1698][FOLLOWUP] fix(test): Adjust jvm opts to increase stability of…"}},{"before":"ac92ae7e1c690a95905f70350de32c55528b0553","after":"a0e88da59f9c7c30d43120c7fa88ec1d0736a616","ref":"refs/heads/master","pushedAt":"2024-05-23T03:27:01.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[MINOR] fix(server): Use warn log level when unable to acquire memory (#1733)\n\n### What changes were proposed in this pull request?\r\n\r\nUse warn log when unable to acquire memory.\r\n\r\n### Why are the changes needed?\r\n\r\nNo need to use error log, because clients will retry for many times.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nNo need.","shortMessageHtmlLink":"[MINOR] fix(server): Use warn log level when unable to acquire memory ("}},{"before":"fddace29d6a7ed4cee69adcb9a656fb62983dc43","after":"ac92ae7e1c690a95905f70350de32c55528b0553","ref":"refs/heads/master","pushedAt":"2024-05-23T01:53:34.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"jerqi","name":"roryqi","path":"/jerqi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8159038?s=80&v=4"},"commit":{"message":"[#1706] improvement: Upgrade the default NodeJS and npm versions of dashboard. (#1707)\n\n### What changes were proposed in this pull request?\r\n\r\nUpgrade the versions of NodeJS and npm.\r\n\r\n### Why are the changes needed?\r\n\r\nThe dashboard on MacOS does not support the M-chip of Node14, resulting in a local compilation failure, and you need to upgrade to Node16 or npm8 or later.\r\n\r\nFix: #1706 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nNo testing required.","shortMessageHtmlLink":"[#1706] improvement: Upgrade the default NodeJS and npm versions of d…"}},{"before":"601119ddab68cc118d6a03e94d0b821899520f87","after":"eaa7af4570eafc42d53cba3c70c0a736edb8082f","ref":"refs/heads/branch-0.9","pushedAt":"2024-05-23T01:52:03.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"jerqi","name":"roryqi","path":"/jerqi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8159038?s=80&v=4"},"commit":{"message":"[#1675][FOLLOWUP] fix(test): Fix various flaky tests (#1730)\n\n### What changes were proposed in this pull request?\r\n\r\nFix various flaky tests:\r\n1. After using `SimpleClusterManager`, release resources at once, which might cause tests to fail sometimes.\r\n2. The compiler machine may not necessarily have a valid broadcast address (maybe in a pod created by K8S, and the broadcast address could be 0.0.0.0). Unit tests should running successfully on such machines. We need to support this scenario.\r\n3. Refactor the `URI.create` logic in `SimpleClusterManagerTest` to support cross-platform operation, such as running tests on Windows.\r\n4. Use `IntegrationTestBase#shutdownServers` to reduce duplicated codes.\r\n\r\n\r\n### Why are the changes needed?\r\n\r\nFor https://github.com/apache/incubator-uniffle/issues/1675.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnnecessary.","shortMessageHtmlLink":"[#1675][FOLLOWUP] fix(test): Fix various flaky tests (#1730)"}},{"before":"00916d87de0e1f69202477d4cd329d2819a64bd3","after":"fddace29d6a7ed4cee69adcb9a656fb62983dc43","ref":"refs/heads/master","pushedAt":"2024-05-23T01:49:51.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"jerqi","name":"roryqi","path":"/jerqi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8159038?s=80&v=4"},"commit":{"message":"[#1675][FOLLOWUP] fix(test): Fix various flaky tests (#1730)\n\n### What changes were proposed in this pull request?\r\n\r\nFix various flaky tests:\r\n1. After using `SimpleClusterManager`, release resources at once, which might cause tests to fail sometimes.\r\n2. The compiler machine may not necessarily have a valid broadcast address (maybe in a pod created by K8S, and the broadcast address could be 0.0.0.0). Unit tests should running successfully on such machines. We need to support this scenario.\r\n3. Refactor the `URI.create` logic in `SimpleClusterManagerTest` to support cross-platform operation, such as running tests on Windows.\r\n4. Use `IntegrationTestBase#shutdownServers` to reduce duplicated codes.\r\n\r\n\r\n### Why are the changes needed?\r\n\r\nFor https://github.com/apache/incubator-uniffle/issues/1675.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnnecessary.","shortMessageHtmlLink":"[#1675][FOLLOWUP] fix(test): Fix various flaky tests (#1730)"}},{"before":"f738a886cd2337eabe128d58edea9e90403e753a","after":"00916d87de0e1f69202477d4cd329d2819a64bd3","ref":"refs/heads/master","pushedAt":"2024-05-23T01:48:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"jerqi","name":"roryqi","path":"/jerqi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8159038?s=80&v=4"},"commit":{"message":"[#1731] improvement(server): Bump gRPC from 1.63.0 to 1.64.0 (#1732)\n\n### What changes were proposed in this pull request?\r\n\r\nBump gRPC from 1.63.0 to 1.64.0.\r\n\r\n### Why are the changes needed?\r\n\r\nFix: https://github.com/apache/incubator-uniffle/issues/1731.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting UTs.","shortMessageHtmlLink":"[#1731] improvement(server): Bump gRPC from 1.63.0 to 1.64.0 (#1732)"}},{"before":"d47ec222f83970c43793b0f75e68bd8015b037a2","after":"f738a886cd2337eabe128d58edea9e90403e753a","ref":"refs/heads/master","pushedAt":"2024-05-22T09:33:08.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1698] fix(test): Increase tests running memory for better stability (#1726)\n\n### What changes were proposed in this pull request?\r\n\r\nIncrease tests running memory for better stability.\r\n\r\n### Why are the changes needed?\r\n\r\nFix: https://github.com/apache/incubator-uniffle/issues/1698.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnnecessary.","shortMessageHtmlLink":"[#1698] fix(test): Increase tests running memory for better stability ("}},{"before":"5dd4eefa7664fd7dd2bd433c48dde2b9f6d4e35f","after":"d47ec222f83970c43793b0f75e68bd8015b037a2","ref":"refs/heads/master","pushedAt":"2024-05-22T09:32:43.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1728] feat(server): Introduce disks timeout metrics (#1729)\n\n### What changes were proposed in this pull request?\r\n\r\nIntroduce disks timeout metrics.\r\n\r\n### Why are the changes needed?\r\n\r\nFor https://github.com/apache/incubator-uniffle/issues/1728.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting tests.","shortMessageHtmlLink":"[#1728] feat(server): Introduce disks timeout metrics (#1729)"}},{"before":"168cf741ae5dfb806edb040b11a6e496c0d73afa","after":"5dd4eefa7664fd7dd2bd433c48dde2b9f6d4e35f","ref":"refs/heads/master","pushedAt":"2024-05-22T07:55:22.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1711] feat(server): Introduce the reconfigurable conf (#1712)\n\n### What changes were proposed in this pull request?\r\n\r\nIntroduce the reconfigurable conf for server or other componts.\r\n\r\n### Why are the changes needed?\r\n\r\nFix: #1711\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnit tests.","shortMessageHtmlLink":"[#1711] feat(server): Introduce the reconfigurable conf (#1712)"}},{"before":"e807f24f7273febf8b50f7c79d6e0c83f36ad6f0","after":"601119ddab68cc118d6a03e94d0b821899520f87","ref":"refs/heads/branch-0.9","pushedAt":"2024-05-22T07:04:16.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[MINOR] fix: Update outdated config: rss.writer.send.check.timeout -> rss.client.send.check.timeout.ms (#1734)\n\n### What changes were proposed in this pull request?\r\n\r\nUpdate outdated config: rss.writer.send.check.timeout -> rss.client.send.check.timeout.ms.\r\n\r\n### Why are the changes needed?\r\n\r\nThe `rss.writer.send.check.timeout` configuration has been renamed to `rss.client.send.check.timeout.ms`, no longer in use.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nNo need.","shortMessageHtmlLink":"[MINOR] fix: Update outdated config: rss.writer.send.check.timeout ->…"}},{"before":"d9b1d9fba58f70347ec2a352f15502eeec53deb9","after":"168cf741ae5dfb806edb040b11a6e496c0d73afa","ref":"refs/heads/master","pushedAt":"2024-05-22T06:53:36.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[MINOR] fix: Update outdated config: rss.writer.send.check.timeout -> rss.client.send.check.timeout.ms (#1734)\n\n### What changes were proposed in this pull request?\r\n\r\nUpdate outdated config: rss.writer.send.check.timeout -> rss.client.send.check.timeout.ms.\r\n\r\n### Why are the changes needed?\r\n\r\nThe `rss.writer.send.check.timeout` configuration has been renamed to `rss.client.send.check.timeout.ms`, no longer in use.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nNo need.","shortMessageHtmlLink":"[MINOR] fix: Update outdated config: rss.writer.send.check.timeout ->…"}},{"before":"c3cbdec4a099efbba0a13d2cc53aafd9493ae498","after":"d9b1d9fba58f70347ec2a352f15502eeec53deb9","ref":"refs/heads/master","pushedAt":"2024-05-21T01:59:22.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#1721] fix(coordinator): classCastExpection of boolean->String with yaml style remote client conf (#1722)\n\n### What changes were proposed in this pull request?\r\n\r\nfix classCastExpection of boolean->String with yaml style remote client conf\r\n\r\n### Why are the changes needed?\r\n\r\nFix: #1721 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nUnit tests","shortMessageHtmlLink":"[#1721] fix(coordinator): classCastExpection of boolean->String with …"}},{"before":null,"after":"88a16337be037777adb53603eb693b75ec6a08a7","ref":"refs/heads/blockid-set","pushedAt":"2024-05-20T09:42:13.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"EnricoMi","name":"Enrico Minack","path":"/EnricoMi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/44700269?s=80&v=4"},"commit":{"message":"Hide implementation of block id set behind BlockIdSet interface","shortMessageHtmlLink":"Hide implementation of block id set behind BlockIdSet interface"}},{"before":"3cc052bc2dae3f90c9b29cc78d1cab303b58b6ed","after":"c3cbdec4a099efbba0a13d2cc53aafd9493ae498","ref":"refs/heads/master","pushedAt":"2024-05-17T12:22:19.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"zuston","name":"Junfan Zhang","path":"/zuston","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8609142?s=80&v=4"},"commit":{"message":"[#519][FOLLOWUP] improvement: Speed up ConcurrentHashMap#computeIfAbsent (#1719)\n\n### What changes were proposed in this pull request?\r\n\r\nSpeed up ConcurrentHashMap#computeIfAbsent by checking key existence.\r\n\r\n### Why are the changes needed?\r\n\r\nA followup pr for https://github.com/apache/incubator-uniffle/issues/519.\r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\n\r\nExisting UTs.","shortMessageHtmlLink":"[#519][FOLLOWUP] improvement: Speed up ConcurrentHashMap#computeIfAbs…"}},{"before":"ab7d0dc5b7609f8eca2af737fa23a28d0e31b6a7","after":"e807f24f7273febf8b50f7c79d6e0c83f36ad6f0","ref":"refs/heads/branch-0.9","pushedAt":"2024-05-17T07:01:31.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"jerqi","name":"roryqi","path":"/jerqi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8159038?s=80&v=4"},"commit":{"message":"[#1673] fix(K8S): Fix the deployment of stable version K8S cluster (#1694)\n\n### What changes were proposed in this pull request?\r\n1. Upgrade Go to 1.19, Because mac brew doesn't support to install older than 1.19 Go.\r\n2. Upgrade K8S api to 0.24. Because 1.24.1 supports to run the tests in the Mac M1 local machine and supports autoscaling/v2 api.\r\n3. Change v2beta2 to v2\r\n4. Use log4j2 instead of log4j\r\n\r\n### Why are the changes needed?\r\nKubernetes 1.26 removed the v2beta2 api of auto scaling. So our operator can't deployed some stable version K8S cluster. So I try to upgrade the version of the client.\r\nCompatible solution will be very difficult and bring more burden to maintain. So I choose to upgrade directly. If some users use low version, they can revert this pull request and compile again.\r\n\r\nFix: #1673 \r\n\r\n### Does this PR introduce _any_ user-facing change?\r\n\r\nNo.\r\n\r\n### How was this patch tested?\r\nUT + Manual test.","shortMessageHtmlLink":"[#1673] fix(K8S): Fix the deployment of stable version K8S cluster (#…"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEXAVykQA","startCursor":null,"endCursor":null}},"title":"Activity · apache/incubator-uniffle"}