I am personally against UUID that does not guarantee at the spec level that
they are unique across something. Even if the spec could guarantee that, it
feels like we are trying to define a type for what should be a constraint.
I would rather remove support for UUID and let the engines do coercion
mat that is designed
>>>>> for high performance and ease of use; and be it further
>>>>>
>>>>> RESOLVED, that the office of "Vice President, Apache Iceberg" be and
>>>>> hereby is created, the person holding such office to serv
+1 once we apply the patch.
The patch works with presto but it breaks iceberg build. We need to add a
typecast to FileIO in TableMetadataParserTest here
<https://github.com/Parth-Brahmbhatt/incubator-iceberg/blob/presto-patch/core/src/test/java/org/apache/iceberg/TableMetadataParserTest.java#
+1(binding)
All checks passed and presto smoke tests pass as well.
On Mon, Oct 21, 2019 at 3:13 PM Daniel Weeks wrote:
> +1
>
> Verified sig, sum, license, build, and tests
>
> On Mon, Oct 21, 2019 at 2:14 PM Ryan Blue
> wrote:
>
>> +1 (binding)
>>
>> Ran release checks, validated metadata