You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@knox.apache.org by GitBox <gi...@apache.org> on 2022/09/02 19:59:58 UTC

[GitHub] [knox] lmccay opened a new pull request, #628: KNOX-2798 - Add a trim method to KnoxShellTable to trim all values in…

lmccay opened a new pull request, #628:
URL: https://github.com/apache/knox/pull/628

   … a Column
   
   ## What changes were proposed in this pull request?
   We need a method to call to trim whitespace from all values in a column of a KnoxShellTable. This is useful for values that are numbers that are padded with spaces in order to enable numeric operations.
   
   ## How was this patch tested?
   
   Added unit tests and manual testing from within KnoxShell
   
   Please review [Knox Contributing Process](https://cwiki.apache.org/confluence/display/KNOX/Contribution+Process#ContributionProcess-GithubWorkflow) before opening a pull request.
   


-- 
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: dev-unsubscribe@knox.apache.org

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


[GitHub] [knox] lmccay commented on pull request #628: KNOX-2798 - Add a trim method to KnoxShellTable to trim all values in…

Posted by GitBox <gi...@apache.org>.
lmccay commented on PR #628:
URL: https://github.com/apache/knox/pull/628#issuecomment-1235909182

   ARM build failure unrelated to change. Merging this PR.


-- 
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: dev-unsubscribe@knox.apache.org

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


[GitHub] [knox] lmccay merged pull request #628: KNOX-2798 - Add a trim method to KnoxShellTable to trim all values in…

Posted by GitBox <gi...@apache.org>.
lmccay merged PR #628:
URL: https://github.com/apache/knox/pull/628


-- 
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: dev-unsubscribe@knox.apache.org

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