You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2020/01/13 13:38:38 UTC

[GitHub] [incubator-hudi] wangxianghu opened a new pull request #1220: [HUDI-397] Normalize log print statement

wangxianghu opened a new pull request #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220
 
 
   ## What is the purpose of the pull request
   
   *Redo hudi-test-suite log statements using SLF4J*
   *Normalize log print statement*
   
   ## Brief change log
   
   *Redo hudi-test-suite log statements using SLF4J*
   *Normalize log print statement*
   
   ## Verify this pull request
   
   This pull request should be covered by existing tests.
   
   ## Committer checklist
   
    - [ ] Has a corresponding JIRA in PR title & commit
    
    - [ ] Commit message is descriptive of the change
    
    - [ ] CI is green
   
    - [ ] Necessary doc changes done or have another open PR
          
    - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] wangxianghu commented on issue #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
wangxianghu commented on issue #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220#issuecomment-573955655
 
 
   Hi @n3nash, This PR covered all the logs in test-suite module. Besides, I found the wrong email was used. I will fix it.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] hmatu commented on issue #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
hmatu commented on issue #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220#issuecomment-573729421
 
 
   If you modify some changes, you can continue to commit to same branch, no need to create a new pr, like https://github.com/apache/incubator-hudi/pull/1219 https://github.com/apache/incubator-hudi/pull/1217 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] wangxianghu commented on issue #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
wangxianghu commented on issue #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220#issuecomment-573953742
 
 
   Hi @hmatu, Thanks for your advice, I will pay attention next 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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] n3nash commented on issue #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
n3nash commented on issue #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220#issuecomment-573811899
 
 
   @wangxianghu Thanks for the PR, does this exhaustively take care of all the logs in the test-suite module ?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] hmatu commented on issue #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
hmatu commented on issue #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220#issuecomment-573730094
 
 
   Please use `git config --list` to check whether your email is right or not.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] wangxianghu commented on issue #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
wangxianghu commented on issue #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220#issuecomment-573958871
 
 
   @hmatu Roll back, It closed automatically

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] wangxianghu closed pull request #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
wangxianghu closed pull request #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220
 
 
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-hudi] hmatu commented on issue #1220: [HUDI-397] Normalize log print statement

Posted by GitBox <gi...@apache.org>.
hmatu commented on issue #1220: [HUDI-397] Normalize log print statement
URL: https://github.com/apache/incubator-hudi/pull/1220#issuecomment-573957902
 
 
   Close again?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services