rhoughton-pivot commented on pull request #5115:
URL: https://github.com/apache/geode/pull/5115#issuecomment-637864800
@Bill I am seeing the following error around the SSL change herein. Can you
advise me on whether I should be putting my class into the `allowed` or
`ignored` file?
```
org.apache.geode.codeAnalysis.AnalyzeCoreSerializablesJUnitTest >
testSerializables FAILED
java.lang.AssertionError: New or moved
classes----------------------------------------
org/apache/geode/net/SSLParameterExtension$SSLParameterExtensionTypes,false
If the class is not persisted or sent over the wire add it to the file
/home/geode/geode/geode-core/build/resources/integrationTest/org/apache/geode/codeAnalysis/excludedClasses.txt
Otherwise if this doesn't break backward compatibility, copy the file
/home/geode/geode/geode-core/build/integrationTest793/actualSerializables.dat
to
sanctioned-geode-core-serializables.txt.
```
----------------------------------------------------------------
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:
us...@infra.apache.org
