You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2020/06/02 23:46:11 UTC

[GitHub] [geode] rhoughton-pivot commented on pull request #5115: Feature/GEODE-7414_2_redo

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:
users@infra.apache.org