prestodb/presto

OutputMatcher should take a Collection rather than a List

Ouverte

#22 756 ouverte le 15 mai 2024

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (1 personne assignée)Java (5 240 forks)batch import
available-to-pickupgood first issueplanner

Métriques du dépôt

Stars
 (15 558 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

com.facebook.presto.sql.planner.assertions.OutputMatcher should take a Collection rather than a List

Order is not important in this class so this is semantically correct, and I encountered a case where it would have been mildly more convenient to pass in a Set instead of a List.

Guide contributeur

Direction de recherche
Localisez la classe OutputMatcher et modifiez son paramètre de méthode de List en Collection. Mettez à jour tous les appelants qui passent une List pour accepter une Collection. Assurez vous qu'aucun autre code ne dépend de l'ordre.
Stack technique
java
Domaine
backenddatabase
Type d'issue
Refactorisation
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.
Moins d'une heure
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
JavaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85