Re: CODEOWNERS? (was Re: Pending PR reviews)

[Patrick Johnson]

+1 for getting rid of CODEOWNERS.

> On Jun 29, 2022, at 9:33 AM, Anthony Baker <bak...@vmware.com.INVALID> wrote:
> 
> ⚠ External Email
> 
> I realize that this is a thread hijack, but hopefully a useful one. I’ve seen 
> several requests for timely reviews in recent months. I think that the 
> CODEOWNERS goals were important and laudable—directing review requests to 
> those most suited to provide oversight—but the implementation has been 
> problematic. The size, complexity, and interconnectedness of the code base 
> meant that many pull requests tagged not just one expert but just about EVERY 
> expert in the community. This is rather inefficient, to say the least.
> 
> I propose that we revert CODEOWNERS and return to the review-then-commit 
> model requiring at least one +1 vote from a committer. I see Owen has already 
> created a PR [1] for this change.
> 
> Thoughts?
> 
> Anthony
> 
> [1] 
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F7820&amp;data=05%7C01%7Cjpatrick%40vmware.com%7Cef39bb9a4e794fba73cd08da59ed3d9a%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637921172911084617%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=PRAvK0HR0cU5Tg59KVD%2BIjnq8PAsaKfjX8%2BG%2FHeDnAw%3D&amp;reserved=0
> 
> 
>> On Jun 28, 2022, at 5:43 AM, Mario Ivanac <mario.iva...@est.tech> wrote:
>> 
>> ⚠ External Email
>> 
>> Hi,
>> 
>> The following PRs:
>> 
>> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F7323&amp;data=05%7C01%7Cjpatrick%40vmware.com%7Cef39bb9a4e794fba73cd08da59ed3d9a%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637921172911084617%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=Tvz9IRs642RD89jY41htcIxuwsm1i4e90BJdmsLBYnI%3D&amp;reserved=0
>> 
>> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F7749&amp;data=05%7C01%7Cjpatrick%40vmware.com%7Cef39bb9a4e794fba73cd08da59ed3d9a%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637921172911084617%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=nv2NsBKWaTDZhe%2BAfiQfM5JslaQDl48Fbk0OrscDWbQ%3D&amp;reserved=0
>> 
>> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F7664&amp;data=05%7C01%7Cjpatrick%40vmware.com%7Cef39bb9a4e794fba73cd08da59ed3d9a%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637921172911084617%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=k9d8Mxjv3llSWWcyL06xjQYDV%2FTXbh393Yk2%2F8%2BOyWU%3D&amp;reserved=0
>> 
>> are waiting for review for some time.
>> 
>> 
>> Could code owners review these PRs?
>> 
>> Thanks,
>> Mario
>> 
>> ________________________________
>> 
>> ⚠ External Email: This email originated from outside of the organization. Do 
>> not click links or open attachments unless you recognize the sender.
>