arvindKandpal-ksolves commented on PR #7971: URL: https://github.com/apache/hbase/pull/7971#issuecomment-4168125503
I've updated this PR based on the feedback: - Cherry-picked the commit from master and amended the commit message to include the standard cherry-pick suffix. - Ran manual tests on a local standalone cluster using the sanity script. When testing the bad syntax (list_namespace, 'ns.*') and variable assignments (list = 10), the shell correctly printed the expected WARN messages and JRuby SyntaxError. - Most importantly, the shell didn't hang or break. All subsequent commands (create, put, scan, etc.) ran smoothly without corrupting the binding. - Automated shell tests (mvn test -Dtest=TestShell -pl hbase-shell -Dhadoop.profile=3.0) also passed locally. This looks ready from my side. Let me know if it's good to merge! I'll start working on the branch-2.5 backport next. cc @NihalJain @Apache9 @liuxiaocs7 @vaijosh -- 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 to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
