You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@iceberg.apache.org by Eduard Tudenhöfner <et...@apache.org> on 2025/11/18 11:44:25 UTC
[VOTE] Adding min-rows-requested to the request of the /plan endpoint
Hey everyone,
I'd like to VOTE on adding an optional *min-rows-requested* field to the
request of the */plan* endpoint so that the client can send a *hint* to the
server to not have to return more rows than necessary. It is not required
for the server to return that many rows since the scan may not produce that
many rows. The server can also return more rows than requested.
The proposed OpenAPI spec changes can be seen in PR #14565
<https://github.com/apache/iceberg/pull/14565>.
*Please vote within the next 72 hours:*
[ ] +1 Add these changes to the spec
[ ] +0
[ ] -1 I have questions and/or concerns
Thanks,
Eduard
Re: [VOTE] Adding min-rows-requested to the request of the /plan endpoint
Posted by Eduard Tudenhöfner <et...@apache.org>.
With 6 +1 votes (4 binding) this passes.
Thanks everyone, I'll get the PR merged.
On Wed, Nov 19, 2025 at 6:25 AM Prashant Singh <pr...@gmail.com>
wrote:
> +1 (non-binding)
>
> Best,
> Prashant
>
> On Wed, Nov 19, 2025 at 4:59 AM huaxin gao <hu...@gmail.com> wrote:
>
>> +1 (non-binding)
>>
>> On Tue, Nov 18, 2025 at 3:07 PM Amogh Jahagirdar <2a...@gmail.com>
>> wrote:
>>
>>> +1
>>>
>>> On Tue, Nov 18, 2025 at 4:00 PM Steven Wu <st...@gmail.com> wrote:
>>>
>>>> +1
>>>>
>>>> On Tue, Nov 18, 2025 at 2:19 PM Fokko Driesprong <fo...@apache.org>
>>>> wrote:
>>>>
>>>>> +1
>>>>>
>>>>> Op di 18 nov 2025 om 22:11 schreef Ryan Blue <rd...@gmail.com>:
>>>>>
>>>>>> Looks good to me. +1
>>>>>>
>>>>>> On Tue, Nov 18, 2025 at 3:44 AM Eduard Tudenhöfner <
>>>>>> etudenhoefner@apache.org> wrote:
>>>>>>
>>>>>>> Hey everyone,
>>>>>>>
>>>>>>> I'd like to VOTE on adding an optional *min-rows-requested* field
>>>>>>> to the request of the */plan* endpoint so that the client can send
>>>>>>> a *hint* to the server to not have to return more rows than
>>>>>>> necessary. It is not required for the server to return that many rows since
>>>>>>> the scan may not produce that many rows. The server can also return more
>>>>>>> rows than requested.
>>>>>>>
>>>>>>> The proposed OpenAPI spec changes can be seen in PR #14565
>>>>>>> <https://github.com/apache/iceberg/pull/14565>.
>>>>>>>
>>>>>>> *Please vote within the next 72 hours:*
>>>>>>>
>>>>>>> [ ] +1 Add these changes to the spec
>>>>>>> [ ] +0
>>>>>>> [ ] -1 I have questions and/or concerns
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Eduard
>>>>>>>
>>>>>>
Re: [VOTE] Adding min-rows-requested to the request of the /plan endpoint
Posted by Prashant Singh <pr...@gmail.com>.
+1 (non-binding)
Best,
Prashant
On Wed, Nov 19, 2025 at 4:59 AM huaxin gao <hu...@gmail.com> wrote:
> +1 (non-binding)
>
> On Tue, Nov 18, 2025 at 3:07 PM Amogh Jahagirdar <2a...@gmail.com> wrote:
>
>> +1
>>
>> On Tue, Nov 18, 2025 at 4:00 PM Steven Wu <st...@gmail.com> wrote:
>>
>>> +1
>>>
>>> On Tue, Nov 18, 2025 at 2:19 PM Fokko Driesprong <fo...@apache.org>
>>> wrote:
>>>
>>>> +1
>>>>
>>>> Op di 18 nov 2025 om 22:11 schreef Ryan Blue <rd...@gmail.com>:
>>>>
>>>>> Looks good to me. +1
>>>>>
>>>>> On Tue, Nov 18, 2025 at 3:44 AM Eduard Tudenhöfner <
>>>>> etudenhoefner@apache.org> wrote:
>>>>>
>>>>>> Hey everyone,
>>>>>>
>>>>>> I'd like to VOTE on adding an optional *min-rows-requested* field to
>>>>>> the request of the */plan* endpoint so that the client can send a
>>>>>> *hint* to the server to not have to return more rows than necessary.
>>>>>> It is not required for the server to return that many rows since the scan
>>>>>> may not produce that many rows. The server can also return more rows than
>>>>>> requested.
>>>>>>
>>>>>> The proposed OpenAPI spec changes can be seen in PR #14565
>>>>>> <https://github.com/apache/iceberg/pull/14565>.
>>>>>>
>>>>>> *Please vote within the next 72 hours:*
>>>>>>
>>>>>> [ ] +1 Add these changes to the spec
>>>>>> [ ] +0
>>>>>> [ ] -1 I have questions and/or concerns
>>>>>>
>>>>>> Thanks,
>>>>>> Eduard
>>>>>>
>>>>>
Re: [VOTE] Adding min-rows-requested to the request of the /plan endpoint
Posted by huaxin gao <hu...@gmail.com>.
+1 (non-binding)
On Tue, Nov 18, 2025 at 3:07 PM Amogh Jahagirdar <2a...@gmail.com> wrote:
> +1
>
> On Tue, Nov 18, 2025 at 4:00 PM Steven Wu <st...@gmail.com> wrote:
>
>> +1
>>
>> On Tue, Nov 18, 2025 at 2:19 PM Fokko Driesprong <fo...@apache.org>
>> wrote:
>>
>>> +1
>>>
>>> Op di 18 nov 2025 om 22:11 schreef Ryan Blue <rd...@gmail.com>:
>>>
>>>> Looks good to me. +1
>>>>
>>>> On Tue, Nov 18, 2025 at 3:44 AM Eduard Tudenhöfner <
>>>> etudenhoefner@apache.org> wrote:
>>>>
>>>>> Hey everyone,
>>>>>
>>>>> I'd like to VOTE on adding an optional *min-rows-requested* field to
>>>>> the request of the */plan* endpoint so that the client can send a
>>>>> *hint* to the server to not have to return more rows than necessary.
>>>>> It is not required for the server to return that many rows since the scan
>>>>> may not produce that many rows. The server can also return more rows than
>>>>> requested.
>>>>>
>>>>> The proposed OpenAPI spec changes can be seen in PR #14565
>>>>> <https://github.com/apache/iceberg/pull/14565>.
>>>>>
>>>>> *Please vote within the next 72 hours:*
>>>>>
>>>>> [ ] +1 Add these changes to the spec
>>>>> [ ] +0
>>>>> [ ] -1 I have questions and/or concerns
>>>>>
>>>>> Thanks,
>>>>> Eduard
>>>>>
>>>>
Re: [VOTE] Adding min-rows-requested to the request of the /plan endpoint
Posted by Amogh Jahagirdar <2a...@gmail.com>.
+1
On Tue, Nov 18, 2025 at 4:00 PM Steven Wu <st...@gmail.com> wrote:
> +1
>
> On Tue, Nov 18, 2025 at 2:19 PM Fokko Driesprong <fo...@apache.org> wrote:
>
>> +1
>>
>> Op di 18 nov 2025 om 22:11 schreef Ryan Blue <rd...@gmail.com>:
>>
>>> Looks good to me. +1
>>>
>>> On Tue, Nov 18, 2025 at 3:44 AM Eduard Tudenhöfner <
>>> etudenhoefner@apache.org> wrote:
>>>
>>>> Hey everyone,
>>>>
>>>> I'd like to VOTE on adding an optional *min-rows-requested* field to
>>>> the request of the */plan* endpoint so that the client can send a
>>>> *hint* to the server to not have to return more rows than necessary.
>>>> It is not required for the server to return that many rows since the scan
>>>> may not produce that many rows. The server can also return more rows than
>>>> requested.
>>>>
>>>> The proposed OpenAPI spec changes can be seen in PR #14565
>>>> <https://github.com/apache/iceberg/pull/14565>.
>>>>
>>>> *Please vote within the next 72 hours:*
>>>>
>>>> [ ] +1 Add these changes to the spec
>>>> [ ] +0
>>>> [ ] -1 I have questions and/or concerns
>>>>
>>>> Thanks,
>>>> Eduard
>>>>
>>>
Re: [VOTE] Adding min-rows-requested to the request of the /plan endpoint
Posted by Steven Wu <st...@gmail.com>.
+1
On Tue, Nov 18, 2025 at 2:19 PM Fokko Driesprong <fo...@apache.org> wrote:
> +1
>
> Op di 18 nov 2025 om 22:11 schreef Ryan Blue <rd...@gmail.com>:
>
>> Looks good to me. +1
>>
>> On Tue, Nov 18, 2025 at 3:44 AM Eduard Tudenhöfner <
>> etudenhoefner@apache.org> wrote:
>>
>>> Hey everyone,
>>>
>>> I'd like to VOTE on adding an optional *min-rows-requested* field to
>>> the request of the */plan* endpoint so that the client can send a *hint* to
>>> the server to not have to return more rows than necessary. It is not
>>> required for the server to return that many rows since the scan may not
>>> produce that many rows. The server can also return more rows than requested.
>>>
>>> The proposed OpenAPI spec changes can be seen in PR #14565
>>> <https://github.com/apache/iceberg/pull/14565>.
>>>
>>> *Please vote within the next 72 hours:*
>>>
>>> [ ] +1 Add these changes to the spec
>>> [ ] +0
>>> [ ] -1 I have questions and/or concerns
>>>
>>> Thanks,
>>> Eduard
>>>
>>
Re: [VOTE] Adding min-rows-requested to the request of the /plan endpoint
Posted by Fokko Driesprong <fo...@apache.org>.
+1
Op di 18 nov 2025 om 22:11 schreef Ryan Blue <rd...@gmail.com>:
> Looks good to me. +1
>
> On Tue, Nov 18, 2025 at 3:44 AM Eduard Tudenhöfner <
> etudenhoefner@apache.org> wrote:
>
>> Hey everyone,
>>
>> I'd like to VOTE on adding an optional *min-rows-requested* field to the
>> request of the */plan* endpoint so that the client can send a *hint* to
>> the server to not have to return more rows than necessary. It is not
>> required for the server to return that many rows since the scan may not
>> produce that many rows. The server can also return more rows than requested.
>>
>> The proposed OpenAPI spec changes can be seen in PR #14565
>> <https://github.com/apache/iceberg/pull/14565>.
>>
>> *Please vote within the next 72 hours:*
>>
>> [ ] +1 Add these changes to the spec
>> [ ] +0
>> [ ] -1 I have questions and/or concerns
>>
>> Thanks,
>> Eduard
>>
>
Re: [VOTE] Adding min-rows-requested to the request of the /plan endpoint
Posted by Ryan Blue <rd...@gmail.com>.
Looks good to me. +1
On Tue, Nov 18, 2025 at 3:44 AM Eduard Tudenhöfner <et...@apache.org>
wrote:
> Hey everyone,
>
> I'd like to VOTE on adding an optional *min-rows-requested* field to the
> request of the */plan* endpoint so that the client can send a *hint* to
> the server to not have to return more rows than necessary. It is not
> required for the server to return that many rows since the scan may not
> produce that many rows. The server can also return more rows than requested.
>
> The proposed OpenAPI spec changes can be seen in PR #14565
> <https://github.com/apache/iceberg/pull/14565>.
>
> *Please vote within the next 72 hours:*
>
> [ ] +1 Add these changes to the spec
> [ ] +0
> [ ] -1 I have questions and/or concerns
>
> Thanks,
> Eduard
>