You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kyuubi.apache.org by GitBox <gi...@apache.org> on 2021/11/25 04:03:58 UTC

[GitHub] [incubator-kyuubi] ulysses-you opened a new issue #1452: [Bug] the new code style has issue when import `org.apache.kyuubi`

ulysses-you opened a new issue #1452:
URL: https://github.com/apache/incubator-kyuubi/issues/1452


   ### 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/incubator-kyuubi/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### Describe the bug
   
   see the snapshot:
   ![image](https://user-images.githubusercontent.com/12025282/143377853-e14d57d8-d20b-4fc4-847a-ee9039fa15d7.png)
   
   more detail see comment https://github.com/apache/incubator-kyuubi/pull/1378/files#r756092002
   
   ### Affects Version(s)
   
   1.4.0
   
   ### Kyuubi Server Log Output
   
   _No response_
   
   ### Kyuubi Engine Log Output
   
   _No response_
   
   ### Kyuubi Server Configurations
   
   _No response_
   
   ### Kyuubi Engine Configurations
   
   _No response_
   
   ### Additional context
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a 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@kyuubi.apache.org

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

[GitHub] [incubator-kyuubi] ulysses-you closed issue #1452: [Bug] the new code style has issue when import `org.apache.kyuubi`

Posted by GitBox <gi...@apache.org>.
ulysses-you closed issue #1452:
URL: https://github.com/apache/incubator-kyuubi/issues/1452


   


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

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

[GitHub] [incubator-kyuubi] ulysses-you commented on issue #1452: [Bug] the new code style has issue when import `org.apache.kyuubi`

Posted by GitBox <gi...@apache.org>.
ulysses-you commented on issue #1452:
URL: https://github.com/apache/incubator-kyuubi/issues/1452#issuecomment-978833301


   have not seen other similar issue


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

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

[GitHub] [incubator-kyuubi] ulysses-you commented on issue #1452: [Bug] the new code style has issue when import `org.apache.kyuubi`

Posted by GitBox <gi...@apache.org>.
ulysses-you commented on issue #1452:
URL: https://github.com/apache/incubator-kyuubi/issues/1452#issuecomment-978814651


   is it possible we can make `import org.apache.kyuubi` to `import org.apache.kyuubi._` automatically  using script ?


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

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

[GitHub] [incubator-kyuubi] pan3793 commented on issue #1452: [Bug] the new code style has issue when import `org.apache.kyuubi`

Posted by GitBox <gi...@apache.org>.
pan3793 commented on issue #1452:
URL: https://github.com/apache/incubator-kyuubi/issues/1452#issuecomment-978820601


   It should be rare to use `import org.apache.kyuubi` in our codebase, do you find it in other place?


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

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

[GitHub] [incubator-kyuubi] pan3793 commented on issue #1452: [Bug] the new code style has issue when import `org.apache.kyuubi`

Posted by GitBox <gi...@apache.org>.
pan3793 commented on issue #1452:
URL: https://github.com/apache/incubator-kyuubi/issues/1452#issuecomment-978811761


   I think @zhouyifan279 has some investigation, scalafmt use regex to match the PREFIX, not the whole import statememt, there is no easy way to solve it. One workaround is using `import org.apache.kyuubi._` to import all things instead of `import org.apache.kyuubi`.


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

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