Re: [PR] [#5932] improve(CLI): Fix the CLI delete the anonymous user [gravitino]

2025-02-06 Thread via GitHub
Abyss-lord commented on PR #6359: URL: https://github.com/apache/gravitino/pull/6359#issuecomment-2639666848 > I agree with Justin that, anonymous user is a system reserved user, should not be deleted, instead of reminding user. @Abyss-lord could you please update the logic here? @sh

Re: [PR] [#5932] improve(CLI): Fix the CLI delete the anonymous user [gravitino]

2025-02-05 Thread via GitHub
shaofengshi commented on PR #6359: URL: https://github.com/apache/gravitino/pull/6359#issuecomment-2638737454 I agree with Justin that, anonymous user is a system reserved user, should not be deleted, instead of reminding user. @Abyss-lord could you please update the logic here? -- This

Re: [PR] [#5932] improve(CLI): Fix the CLI delete the anonymous user [gravitino]

2025-01-23 Thread via GitHub
tengqm commented on PR #6359: URL: https://github.com/apache/gravitino/pull/6359#issuecomment-2609649732 Sorry, I don't get the merit of this change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] [#5932] improve(CLI): Fix the CLI delete the anonymous user [gravitino]

2025-01-22 Thread via GitHub
Abyss-lord commented on PR #6359: URL: https://github.com/apache/gravitino/pull/6359#issuecomment-2608782103 Hi @justinmclean @tengqm , could you please review this PR when you have time? I’d really appreciate your feedback. -- This is an automated message from the Apache Git Service. To