dcapwell commented on code in PR #2151:
URL: https://github.com/apache/cassandra/pull/2151#discussion_r1103166576


##########
ide/idea/workspace.xml:
##########
@@ -173,7 +183,28 @@
       <option name="MAIN_CLASS_NAME" value="" />
       <option name="METHOD_NAME" value="" />
       <option name="TEST_OBJECT" value="class" />
-      <option name="VM_PARAMETERS" 
value="-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml 
-Dlogback.configurationFile=file://$PROJECT_DIR$/test/conf/logback-test.xml 
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs 
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.skip_sync=true -ea 
-XX:MaxMetaspaceSize=1G -XX:SoftRefLRUPolicyMSPerMB=0 
-XX:HeapDumpPath=build/test -XX:ActiveProcessorCount=2 
-Dcassandra.strict.runtime.checks=true 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dmigration-sstable-root=$PROJECT_DIR$/test/data/migration-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.tolerate_sstable_size=true 
-Dcassandra.skip_sync=true 
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enab
 led=true -Dcassandra.use_nix_recursive_delete=true 
-Dcassandra.test.messagingService.nonGracefulShutdown=true 
-Dcassandra.test.flush_local_schema_changes=false " />
+      <option name="VM_PARAMETERS" value="
+                                          
-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml
+                                          
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs
+                                          
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true
+                                          -Dcassandra.ring_delay_ms=1000
+                                          -Dcassandra.skip_sync=true

Review Comment:
   thanks for dedupping =)



##########
ide/idea/workspace.xml:
##########
@@ -173,7 +183,28 @@
       <option name="MAIN_CLASS_NAME" value="" />
       <option name="METHOD_NAME" value="" />
       <option name="TEST_OBJECT" value="class" />
-      <option name="VM_PARAMETERS" 
value="-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml 
-Dlogback.configurationFile=file://$PROJECT_DIR$/test/conf/logback-test.xml 
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs 
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.skip_sync=true -ea 
-XX:MaxMetaspaceSize=1G -XX:SoftRefLRUPolicyMSPerMB=0 
-XX:HeapDumpPath=build/test -XX:ActiveProcessorCount=2 
-Dcassandra.strict.runtime.checks=true 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dmigration-sstable-root=$PROJECT_DIR$/test/data/migration-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.tolerate_sstable_size=true 
-Dcassandra.skip_sync=true 
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enab
 led=true -Dcassandra.use_nix_recursive_delete=true 
-Dcassandra.test.messagingService.nonGracefulShutdown=true 
-Dcassandra.test.flush_local_schema_changes=false " />
+      <option name="VM_PARAMETERS" value="
+                                          
-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml
+                                          
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs
+                                          
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true
+                                          -Dcassandra.ring_delay_ms=1000
+                                          -Dcassandra.skip_sync=true
+                                          
-Dcassandra.strict.runtime.checks=true
+                                          
-Dcassandra.test.flush_local_schema_changes=false
+                                          
-Dcassandra.test.messagingService.nonGracefulShutdown=true
+                                          
-Dcassandra.tolerate_sstable_size=true
+                                          
-Dcassandra.use_nix_recursive_delete=true
+                                          
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables
+                                          
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin
+                                          
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables

Review Comment:
   thanks for dedupping =)



##########
ide/idea/workspace.xml:
##########
@@ -193,7 +224,18 @@
     <configuration default="false" name="Cassandra" type="Application" 
factoryName="Application">
       <extension name="coverage" enabled="false" merge="false" 
sample_coverage="true" runner="idea" />
       <option name="MAIN_CLASS_NAME" 
value="org.apache.cassandra.service.CassandraDaemon" />
-      <option name="VM_PARAMETERS" value="-Dcassandra-foreground=yes 
-Dcassandra.config=file://$PROJECT_DIR$/conf/cassandra.yaml 
-Dcassandra.storagedir=$PROJECT_DIR$/data 
-Dlogback.configurationFile=file://$PROJECT_DIR$/conf/logback.xml 
-Dcassandra.logdir=$PROJECT_DIR$/data/logs 
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin -Dcassandra.jmx.local.port=7199 
-ea -Xmx1G 
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true 
-XX:HeapDumpPath=build/test" />
+      <option name="VM_PARAMETERS" value="
+                                          -Dcassandra-foreground=yes
+                                          
-Dcassandra.config=file://$PROJECT_DIR$/conf/cassandra.yaml
+                                          -Dcassandra.jmx.local.port=7199
+                                          
-Dcassandra.logdir=$PROJECT_DIR$/data/logs
+                                          
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true
+                                          
-Dcassandra.storagedir=$PROJECT_DIR$/data
+                                          
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin
+                                          
-Dlogback.configurationFile=file://$PROJECT_DIR$/conf/logback.xml
+                                          -Xmx1G
+                                          -ea

Review Comment:
   why did you remove `-XX:HeapDumpPath=build/test`?



##########
ide/idea/workspace.xml:
##########
@@ -173,7 +183,28 @@
       <option name="MAIN_CLASS_NAME" value="" />
       <option name="METHOD_NAME" value="" />
       <option name="TEST_OBJECT" value="class" />
-      <option name="VM_PARAMETERS" 
value="-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml 
-Dlogback.configurationFile=file://$PROJECT_DIR$/test/conf/logback-test.xml 
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs 
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.skip_sync=true -ea 
-XX:MaxMetaspaceSize=1G -XX:SoftRefLRUPolicyMSPerMB=0 
-XX:HeapDumpPath=build/test -XX:ActiveProcessorCount=2 
-Dcassandra.strict.runtime.checks=true 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dmigration-sstable-root=$PROJECT_DIR$/test/data/migration-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.tolerate_sstable_size=true 
-Dcassandra.skip_sync=true 
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enab
 led=true -Dcassandra.use_nix_recursive_delete=true 
-Dcassandra.test.messagingService.nonGracefulShutdown=true 
-Dcassandra.test.flush_local_schema_changes=false " />
+      <option name="VM_PARAMETERS" value="
+                                          
-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml
+                                          
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs
+                                          
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true
+                                          -Dcassandra.ring_delay_ms=1000

Review Comment:
   thanks for dedupping =)



##########
ide/idea/workspace.xml:
##########
@@ -173,7 +183,28 @@
       <option name="MAIN_CLASS_NAME" value="" />
       <option name="METHOD_NAME" value="" />
       <option name="TEST_OBJECT" value="class" />
-      <option name="VM_PARAMETERS" 
value="-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml 
-Dlogback.configurationFile=file://$PROJECT_DIR$/test/conf/logback-test.xml 
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs 
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.skip_sync=true -ea 
-XX:MaxMetaspaceSize=1G -XX:SoftRefLRUPolicyMSPerMB=0 
-XX:HeapDumpPath=build/test -XX:ActiveProcessorCount=2 
-Dcassandra.strict.runtime.checks=true 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dmigration-sstable-root=$PROJECT_DIR$/test/data/migration-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.tolerate_sstable_size=true 
-Dcassandra.skip_sync=true 
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enab
 led=true -Dcassandra.use_nix_recursive_delete=true 
-Dcassandra.test.messagingService.nonGracefulShutdown=true 
-Dcassandra.test.flush_local_schema_changes=false " />
+      <option name="VM_PARAMETERS" value="
+                                          
-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml
+                                          
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs
+                                          
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true
+                                          -Dcassandra.ring_delay_ms=1000
+                                          -Dcassandra.skip_sync=true
+                                          
-Dcassandra.strict.runtime.checks=true
+                                          
-Dcassandra.test.flush_local_schema_changes=false
+                                          
-Dcassandra.test.messagingService.nonGracefulShutdown=true
+                                          
-Dcassandra.tolerate_sstable_size=true
+                                          
-Dcassandra.use_nix_recursive_delete=true
+                                          
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables

Review Comment:
   thanks for dedupping =)



##########
ide/idea/workspace.xml:
##########
@@ -143,7 +143,17 @@
     <configuration default="true" type="Application" factoryName="Application">
       <extension name="coverage" enabled="false" merge="false" 
sample_coverage="true" runner="idea" />
       <option name="MAIN_CLASS_NAME" value="" />
-      <option name="VM_PARAMETERS" 
value="-Dcassandra.config=file://$PROJECT_DIR$/conf/cassandra.yaml 
-Dcassandra.storagedir=$PROJECT_DIR$/data 
-Dlogback.configurationFile=file://$PROJECT_DIR$/conf/logback.xml 
-Dcassandra.logdir=$PROJECT_DIR$/data/logs 
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin -DQT_SHRINKS=0 -ea 
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true 
-XX:HeapDumpPath=build/test" />
+      <option name="VM_PARAMETERS" value="
+                                          -DQT_SHRINKS=0
+                                          
-Dcassandra.config=file://$PROJECT_DIR$/conf/cassandra.yaml
+                                          
-Dcassandra.logdir=$PROJECT_DIR$/data/logs
+                                          
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true
+                                          
-Dcassandra.storagedir=$PROJECT_DIR$/data
+                                          
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin
+                                          
-Dlogback.configurationFile=file://$PROJECT_DIR$/conf/logback.xml
+                                          -ea
+                                          -XX:HeapDumpPath=build/test
+                                          
-Didea.click.expand.for.rest.of.options" />

Review Comment:
   this is new



##########
ide/idea/workspace.xml:
##########
@@ -173,7 +183,28 @@
       <option name="MAIN_CLASS_NAME" value="" />
       <option name="METHOD_NAME" value="" />
       <option name="TEST_OBJECT" value="class" />
-      <option name="VM_PARAMETERS" 
value="-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml 
-Dlogback.configurationFile=file://$PROJECT_DIR$/test/conf/logback-test.xml 
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs 
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.skip_sync=true -ea 
-XX:MaxMetaspaceSize=1G -XX:SoftRefLRUPolicyMSPerMB=0 
-XX:HeapDumpPath=build/test -XX:ActiveProcessorCount=2 
-Dcassandra.strict.runtime.checks=true 
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables 
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables 
-Dmigration-sstable-root=$PROJECT_DIR$/test/data/migration-sstables 
-Dcassandra.ring_delay_ms=1000 -Dcassandra.tolerate_sstable_size=true 
-Dcassandra.skip_sync=true 
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enab
 led=true -Dcassandra.use_nix_recursive_delete=true 
-Dcassandra.test.messagingService.nonGracefulShutdown=true 
-Dcassandra.test.flush_local_schema_changes=false " />
+      <option name="VM_PARAMETERS" value="
+                                          
-Dcassandra.config=file://$PROJECT_DIR$/test/conf/cassandra.yaml
+                                          
-Dcassandra.logdir=$PROJECT_DIR$/build/test/logs
+                                          
-Dcassandra.reads.thresholds.coordinator.defensive_checks_enabled=true
+                                          -Dcassandra.ring_delay_ms=1000
+                                          -Dcassandra.skip_sync=true
+                                          
-Dcassandra.strict.runtime.checks=true
+                                          
-Dcassandra.test.flush_local_schema_changes=false
+                                          
-Dcassandra.test.messagingService.nonGracefulShutdown=true
+                                          
-Dcassandra.tolerate_sstable_size=true
+                                          
-Dcassandra.use_nix_recursive_delete=true
+                                          
-Dinvalid-legacy-sstable-root=$PROJECT_DIR$/test/data/invalid-legacy-sstables
+                                          
-Djava.library.path=$PROJECT_DIR$/lib/sigar-bin
+                                          
-Dlegacy-sstable-root=$PROJECT_DIR$/test/data/legacy-sstables
+                                          
-Dlogback.configurationFile=file://$PROJECT_DIR$/test/conf/logback-test.xml
+                                          
-Dmigration-sstable-root=$PROJECT_DIR$/test/data/migration-sstables
+                                          -XX:ActiveProcessorCount=2
+                                          -XX:HeapDumpPath=build/test
+                                          -XX:MaxMetaspaceSize=1G
+                                          -XX:SoftRefLRUPolicyMSPerMB=0
+                                          -ea
+                                          
-Didea.click.expand.for.rest.of.options" />

Review Comment:
   this one is new... quick google and didn't find results, what is this?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to