[ 
https://issues.apache.org/jira/browse/BROOKLYN-162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660253#comment-14660253
 ] 

ASF GitHub Bot commented on BROOKLYN-162:
-----------------------------------------

Github user aledsage commented on a diff in the pull request:

    https://github.com/apache/incubator-brooklyn/pull/800#discussion_r36435392
  
    --- Diff: 
examples/simple-nosql-cluster/src/main/java/org/apache/brooklyn/demo/CumulusRDFApplication.java
 ---
    @@ -138,8 +139,8 @@ public void initApp() {
                 cassandra = addChild(EntitySpec.create(CassandraFabric.class)
                         .configure(CassandraDatacenter.CLUSTER_NAME, 
"Brooklyn")
                         .configure(CassandraDatacenter.INITIAL_SIZE, 
getConfig(CASSANDRA_CLUSTER_SIZE)) // per location
    -                    .configure(CassandraDatacenter.ENDPOINT_SNITCH_NAME, 
"brooklyn.entity.nosql.cassandra.customsnitch.MultiCloudSnitch")
    -                    .configure(CassandraNode.CUSTOM_SNITCH_JAR_URL, 
"classpath://brooklyn/entity/nosql/cassandra/cassandra-multicloud-snitch.jar")
    +                    .configure(CassandraDatacenter.ENDPOINT_SNITCH_NAME, 
"org.apache.brooklyn.entity.nosql.cassandra.customsnitch.MultiCloudSnitch")
    --- End diff --
    
    Note that this will require us re-building the jar for MultiCloudSnitch 
(which shouldn't even be in brooklyn!).
    
    See the comment at 
https://github.com/apache/incubator-brooklyn/pull/802#issuecomment-128424637
    
    Also see docs/_extra/big_examples/nosql-cassandra/cassandra.include.md 
lines 191 and 205, where the `MultiCloudSnitch` is mentioned. We should do a 
global search for "MultiCloudSnitch".
    
    Don't worry about it here. Let's fix it by moving MultiCloudSnitch 
somewhere more sensible in a completely different PR!


> Update package names to org.apache.brooklyn.*
> ---------------------------------------------
>
>                 Key: BROOKLYN-162
>                 URL: https://issues.apache.org/jira/browse/BROOKLYN-162
>             Project: Brooklyn
>          Issue Type: Task
>    Affects Versions: 0.7.0
>            Reporter: Hadrian Zbarcea
>             Fix For: 0.8.0
>
>
> As discussed on the list we should update the package names sooner rather 
> than later. This issue is meant to track the progress of that task. To 
> minimize interference I added the list of subprojects. Please add your name 
> to the module you want to take care of so others would know you are taking 
> care of it. I'd suggest adding a check mark to what's already done and 
> committed (v) and a (userid) for wip.
> {noformat}
> (hadrian) ./api/
>  ./camp/camp-base/
>  ./camp/camp-server/
>  ./camp/
>  ./core/
>  ./core/src/test/dependencies/osgi/entities/
>  ./core/src/test/dependencies/osgi/more-entities-v1/
>  ./core/src/test/dependencies/osgi/more-entities-v2-evil-twin/
>  ./core/src/test/dependencies/osgi/more-entities-v2/
>  ./examples/global-web-fabric/
> (v)  ./examples/
> (v)  ./examples/simple-messaging-pubsub/
> (v)  ./examples/simple-nosql-cluster/
> (v)  ./examples/simple-web-cluster/
> (v)  ./examples/webapps/hello-world-sql/
> (v)  ./examples/webapps/hello-world-webapp/
> (v)  ./examples/webapps/
>  ./locations/jclouds/
> (v) ./parent/
>  ./policy/
>  ./
> (aledsage)  ./sandbox/cassandra-multicloud-snitch/
> (aledsage)  ./sandbox/database/
> (aledsage)  ./sandbox/extra/
> (aledsage)  ./sandbox/mobile-app/
> (aledsage)  ./sandbox/monitoring/
> (aledsage)  ./sandbox/nosql/
>  ./software/base/
>  ./software/database/
>  ./software/messaging/
>  ./software/monitoring/
>  ./software/network/
>  ./software/nosql/
>  ./software/osgi/
>  ./software/webapp/
>  ./storage/hazelcast/
>  ./usage/all/
>  ./usage/archetypes/quickstart/
>  ./usage/archetypes/quickstart/src/brooklyn-sample/
>  ./usage/archetypes/quickstart/src/main/resources/archetype-resources/
> ./usage/archetypes/quickstart/src/test/resources/projects/integration-test-1/reference/
>  ./usage/archetypes/quickstart/target/classes/archetype-resources/
>  
> ./usage/archetypes/quickstart/target/test-classes/projects/integration-test-1/project/brooklyn-sample/
>  
> ./usage/archetypes/quickstart/target/test-classes/projects/integration-test-1/reference/
> (v)  ./usage/camp/
> (v)  ./usage/cli/
> (v) ./usage/dist/
>  ./usage/downstream-parent/
>  ./usage/jsgui/
>  ./usage/launcher/
>  ./usage/logback-includes/
>  ./usage/logback-xml/
> (v)  ./usage/qa/
>  ./usage/rest-api/
>  ./usage/rest-client/
>  ./usage/rest-server/
>  ./usage/test-support/
>  ./utils/common/
>  ./utils/groovy/
>  ./utils/jmx/jmxmp-ssl-agent/
>  ./utils/jmx/jmxrmi-agent/
>  ./utils/rest-swagger/
>  ./utils/test-support/
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to