keksmd commented on PR #961: URL: https://github.com/apache/incubator-graphar/pull/961#issuecomment-5397107105
@SemyonSinchenko Спасибо за второй проход — сверил весь набор review-комментариев и поправил оставшиеся два minor пункта в `fa967da`: 1. `ColumnType.isScalar` теперь исчерпывающий `switch`: новый `Kind` нельзя молча принять как scalar; unhandled value падает явно. 2. Добавил value semantics для `Schema` и `WriteRequest` (`equals`/`hashCode`) и тесты на одинаковые независимые schema/request. Предыдущие пять замечаний из первого review также остаются закрытыми: value `Field`, duplicate schema names, vector `RecordBatch`, recursive Arrow-shaped `ColumnType`, и `APPEND`. Проверка: `mvn --no-transfer-progress -q -f maven-projects/pom.xml -pl io-api -am verify` — 35 tests, 0 failures; Spotless, pre-commit и `git diff --check` прошли. Когда будет минутка, пожалуйста, посмотри ещё раз и, если всё ок, approve. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
