StarRocks, a Linux Foundation project, is a next-generation sub-second MPP OLAP database for full analytics scenarios, including multi-dimensional analytics, real-time analytics, and ad-hoc queries. StarRocks received InfoWorld’s 2023 BOSSIE Award for best open source software.

StarRocks

Hi everyone! could I please get some eyes on this: <https://github.com/StarRocks/starrocks/pull/71749|github.com/StarRocks/starrocks/pull/71749> it has been stalled since I addressed the comments in July. :slightly_smiling_face: Thank you so much in advance!!

the code conflicts the current main branch, need a rebase

<@U03AB36PTL7> rebased it now! would love it if you/the team are able to take a look :slightly_smiling_face:

hey <@U03AB36PTL7> anything else needed from my side? the PR has passed all checks except failed to provision an ECI instance for testing due to capacity issues from AWS.

looks like the pipeline was a success <@U03AB36PTL7> great :slightly_smiling_face: let me know if there is anything more you need from me ! happy to help

<@U03AB36PTL7> it seems like you have approved the changes which is awesome! is the PR waiting on anything else before it can be merged ? :slightly_smiling_face:

<@U03AB36PTL7> I just saw that the related issue is closed here <https://github.com/StarRocks/starrocks/issues/71547#event-34901622980|github.com/StarRocks/starrocks/issues/71547#…> do you mind reopening? I am also wondering if anything else is holding up the PR I see it needs another approval on top of yours. Can I do anything to follow up with anyone else to help get this merged :slightly_smiling_face:

The PR conflicts with the main branch, needs a rebase and resolve the conflict before it can be merged.

hi <@U03AB36PTL7> thanks for the response I can see that. I have rebased it again, can you please approve again, keen to get it merged this time :slightly_smiling_face: I think we need another approver from your side. Thanks for reopening the related issue!

thanks a lot for helping to get this merged!!