prestodb/presto

OutputMatcher should take a Collection rather than a List

Offen

#22.756 geöffnet am 15.05.2024

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Java (5.240 Forks)batch import
available-to-pickupgood first issueplanner

Repository-Metriken

Stars
 (15.558 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Finde die OutputMatcher Klasse und ändere ihren Methodenparameter von List zu Collection. Aktualisiere alle Aufrufer, die eine List übergeben, sodass sie eine Collection akzeptieren. Stelle sicher, dass kein anderer Code von der Reihenfolge abhängt.
Tech Stack
java
Domain
backenddatabase
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85