apache/pinot

Flaky Test: PartialUpsertTableRebalanceIntegrationTest

Fermée

#12 412 ouverte le 13 févr. 2024

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Java (1 234 forks)batch import
flaky-testhelp wanted

Métriques du dépôt

Stars
 (4 937 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Seeing this occasionally. Maybe 60s is not enough for the rebalance to converge.

Error:  Tests run: 5, Failures: 1, Errors: 0, Skipped: 3, Time elapsed: 86.82 s <<< FAILURE! -- in org.apache.pinot.integration.tests.PartialUpsertTableRebalanceIntegrationTest
Error:  org.apache.pinot.integration.tests.PartialUpsertTableRebalanceIntegrationTest.testRebalance -- Time elapsed: 10.94 s
Error:  org.apache.pinot.integration.tests.PartialUpsertTableRebalanceIntegrationTest.afterMethod -- Time elapsed: 70.99 s <<< FAILURE!
java.lang.AssertionError: Failed to meet condition in 60000ms, error message: Failed to drop the segments
	at org.testng.Assert.fail(Assert.java:111)
	at org.apache.pinot.util.TestUtils.waitForCondition(TestUtils.java:111)
	at org.apache.pinot.util.TestUtils.waitForCondition(TestUtils.java:88)
	at org.apache.pinot.util.TestUtils.waitForCondition(TestUtils.java:93)
	at org.apache.pinot.integration.tests.PartialUpsertTableRebalanceIntegrationTest.afterMethod(PartialUpsertTableRebalanceIntegrationTest.java:192)

Guide contributeur

Direction de recherche
Le problème décrit un test instable où un délai d'attente de 60 secondes est insuffisant. Pour le corriger, augmentez le délai d'attente dans l'appel `TestUtils.waitForCondition` dans `afterMethod` de `PartialUpsertTableRebalanceIntegrationTest.java` ou envisagez d'ajouter une boucle de réessai. Assurez vous que le test réussisse systématiquement en l'exécutant plusieurs fois localement.
Stack technique
java
Domaine
databasebackend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaTesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60