JiaLiangC opened a new pull request, #3859:
URL: https://github.com/apache/ambari/pull/3859

   ## What changes were proposed in this pull request?
   
   ## Problem
   After upgrading from JDK 8 to JDK 17, the 
`testAllPropertiesHaveMarkdownDescriptions` test in `ConfigurationTest` was 
failing due to an `AnnotationFormatError`. This error occurred because 
PowerMock was modifying the `ClusterScale` annotation at runtime, adding an 
additional interface that conflicted with JDK 17's stricter annotation 
processing.
   
   ## Solution
   To resolve this issue, we have moved the `ConfigurationMarkdown` and 
`ClusterScale` annotations to separate files. This prevents PowerMock from 
modifying these specific annotations during test execution, allowing the test 
to pass successfully.
   
   ## Changes
   - Created new files for `ConfigurationMarkdown` and `ClusterScale` 
annotations
   - Updated import statements where necessary
   - No functional changes to the annotations or their usage
   
   ## Testing
   - Verified that the `testAllPropertiesHaveMarkdownDescriptions` test now 
passes
   
![image](https://github.com/user-attachments/assets/2669aefb-7c36-480c-a796-b0f5613d40cf)
   
   ## Notes
   This change highlights a potential incompatibility between PowerMock and JDK 
17 when dealing with certain annotations. We may need to consider alternatives 
to PowerMock or update our testing strategy for future JDK upgrades.
   
   
   Please review [Ambari Contributing 
Guide](https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute) 
before opening a pull request.


-- 
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