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 w
tandard at-rest table format 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 ho
+1. checks passed, did not observe the unit test failure.
Thanks
Parth
On Thu, Apr 30, 2020 at 9:13 AM Daniel Weeks wrote:
> +1 all checks passed
>
> On Thu, Apr 30, 2020 at 8:53 AM Anton Okolnychyi
> wrote:
>
>> That test uses many concurrent writes and I’ve seen cases when it led to
>> deadl
+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 ta
