OutputMatcher should take a Collection rather than a List · prestodb/presto#22756

(3 留言) (0 反應) (1 負責人)Java (15,558 star) (5,240 fork)batch import

available-to-pickupgood first issueplanner

描述

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.

貢獻者指南

技術棧: java
領域: database
議題類型: refactor
難度: 2
預計時間: under 1 hour
活動狀態: blocked
清晰度: clear
前置要求: JavaPresto codebase basics
新手友善度: 20
研究方向: Locate the class OutputMatcher in the package com.facebook.presto.sql.planner.assertions. The goal is to change the method parameter type from List to Collection. Ensure that the change does not break any existing code or tests. Review the existing tests for OutputMatcher to confirm they still pass. Since an assignee is already set, check if there is any ongoing work or PRs linked to this issue.