You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "eric-maynard (via GitHub)" <gi...@apache.org> on 2025/02/17 21:40:06 UTC

[I] Split packages [polaris]

eric-maynard opened a new issue, #1017:
URL: https://github.com/apache/polaris/issues/1017

   ### Describe the bug
   
   When building main, I notice many warnings about split packages. e.g.:
   ```
   $ git checkout upstream/main
   Note: switching to 'upstream/main'.
   
   You are in 'detached HEAD' state. You can look around, make experimental
   changes and commit them, and you can discard any commits you make in this
   state without impacting any branches by switching back to a branch.
   
   If you want to create a new branch to retain commits you create, you may
   do so (now or later) by using -c with the switch command. Example:
   
     git switch -c <new-branch-name>
   
   Or undo this operation with:
   
     git switch -
   
   Turn off this advice by setting config variable advice.detachedHead to false
   
   HEAD is now at 47aace13 Use and update DISCLAIMER (instead of DISCLAIMER_WIP) and cleanup NOTICE (#998)
   
   $ ./gradlew :polaris-quarkus-service:test
   Configuration on demand is an incubating feature.
   Detected a split package usage which is considered a bad practice and should be avoided. Following packages were detected in multiple archives: 
   - "org.apache.polaris.service.quarkus.catalog.io" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.metrics" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.legacy" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.persistence" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.storage" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.config" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.tracing" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.logging" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.context" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.ratelimiter" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.auth" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   - "org.apache.polaris.service.quarkus.task" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   Detected a split package usage which is considered a bad practice and should be avoided. Following packages were detected in multiple archives: 
   - "org.apache.polaris.service.quarkus.catalog.io" found in [org.apache.polaris:polaris-quarkus-service:1.0.0-incubating-SNAPSHOT]
   . . .
   ```
   
   ### To Reproduce
   
   git checkout main
   ./gradlew :polaris-quarkus-service:test
   
   ### Actual Behavior
   
   _No response_
   
   ### Expected Behavior
   
   _No response_
   
   ### Additional context
   
   _No response_
   
   ### System information
   
   _No response_


-- 
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: issues-unsubscribe@polaris.apache.org.apache.org

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


Re: [I] Split packages [polaris]

Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on issue #1017:
URL: https://github.com/apache/polaris/issues/1017#issuecomment-2665226894

   Yes this is annoying but I think it's all false positives. We need to report this to the Quarkus team but I don't have time right now to come up with a simple reproducer.


-- 
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: issues-unsubscribe@polaris.apache.org

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


Re: [I] Split packages [polaris]

Posted by "eric-maynard (via GitHub)" <gi...@apache.org>.
eric-maynard commented on issue #1017:
URL: https://github.com/apache/polaris/issues/1017#issuecomment-2664117380

   cc @adutra as the author of #849 


-- 
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: issues-unsubscribe@polaris.apache.org

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


Re: [I] Split packages [polaris]

Posted by "eric-maynard (via GitHub)" <gi...@apache.org>.
eric-maynard closed issue #1017: Split packages
URL: https://github.com/apache/polaris/issues/1017


-- 
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: issues-unsubscribe@polaris.apache.org

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