You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by "yaooqinn (via GitHub)" <gi...@apache.org> on 2023/09/05 08:31:46 UTC

[GitHub] [kyuubi] yaooqinn opened a new issue, #5255: [Improvement] Add an optional comment field to the authz specs for better recognition

yaooqinn opened a new issue, #5255:
URL: https://github.com/apache/kyuubi/issues/5255

   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   
   
   ### Search before asking
   
   - [X] I have searched in the [issues](https://github.com/apache/kyuubi/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### What would you like to be improved?
   
   Some plans may come from third parties, and we can provide some comment there
   
   ### How should we improve?
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [ ] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
   - [X] No. I cannot submit a PR at this time.


-- 
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: notifications-unsubscribe@kyuubi.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


Re: [I] [TASK][EASY] Add an optional comment field to the authz specs for better recognition [kyuubi]

Posted by "davidyuan1223 (via GitHub)" <gi...@apache.org>.
davidyuan1223 commented on issue #5255:
URL: https://github.com/apache/kyuubi/issues/5255#issuecomment-1797284212

   > The command descs are curated in tests, so it's hard for users to get such information in runtime, especially encountering errors. I‘d like to add an extra field called 'comment' to these descs, it can be used as a part of the error messages and log for debugging.
   
   get, i think i can fix


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


Re: [I] [TASK][EASY] Add an optional comment field to the authz specs for better recognition [kyuubi]

Posted by "yaooqinn (via GitHub)" <gi...@apache.org>.
yaooqinn closed issue #5255: [TASK][EASY] Add an optional comment field to the authz specs for better recognition 
URL: https://github.com/apache/kyuubi/issues/5255


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


Re: [I] [TASK][EASY] Add an optional comment field to the authz specs for better recognition [kyuubi]

Posted by "yaooqinn (via GitHub)" <gi...@apache.org>.
yaooqinn commented on issue #5255:
URL: https://github.com/apache/kyuubi/issues/5255#issuecomment-1797158072

   The command descs are curated in tests, so it's hard for users to get such information in runtime, especially encounting errors. I‘d like to add an extra field called 'comment' to these descs, it can be used as a part of the error messages and loggge for debugging.


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


Re: [I] [TASK][EASY] Add an optional comment field to the authz specs for better recognition [kyuubi]

Posted by "yaooqinn (via GitHub)" <gi...@apache.org>.
yaooqinn commented on issue #5255:
URL: https://github.com/apache/kyuubi/issues/5255#issuecomment-1794619909

   What other information would you like me to provide? 


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


Re: [I] [TASK][EASY] Add an optional comment field to the authz specs for better recognition [kyuubi]

Posted by "davidyuan1223 (via GitHub)" <gi...@apache.org>.
davidyuan1223 commented on issue #5255:
URL: https://github.com/apache/kyuubi/issues/5255#issuecomment-1794624747

   > What other information would you like me to provide?
   
   my problem, i want know what we want to provide comment, i found there only extractor's implementations have special third parties,we wanna provide comment for the special extractor?


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


Re: [I] [TASK][EASY] Add an optional comment field to the authz specs for better recognition [kyuubi]

Posted by "davidyuan1223 (via GitHub)" <gi...@apache.org>.
davidyuan1223 commented on issue #5255:
URL: https://github.com/apache/kyuubi/issues/5255#issuecomment-1794596977

   looks this issue i can fix, and could you desc more info?


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org