- 
                Notifications
    You must be signed in to change notification settings 
- Fork 3.4k
HBASE-28368 Backport "HBASE-27693 Support for Hadoop's LDAP Authentic… #5680
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| The test  | 
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| The failure for jdk8 hadoop2 is related? | 
| Yes I have checked the test failures are related. There's more changes needed for hadoop2. In hadoop 2 the version of directory services used is different from what we are importing for hadoop 3 and both versions have somewhat incompatible changes. I will submit a fix to make test run for both. | 
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| The failure precommit checks / yetus jdk8 Hadoop2 checks / org.apache.hadoop.hbase.http.log.TestLogLevel.(?) is related: We see this as we have overridden the apacheds version in pom for all hadoop versions but for hadoop 2 it fails to start KDC as the above package does not even exist in apacheds 2.0.0.AM26 while hadoop 2 requires 2.0.0-M15. So as a fix I propose to skip running LDAP tests fro hadoop2 and also moving all dependency changes to be only for hadoop3. | 
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| Failures are not related, previous build test failure issue is resolved | 
| <artifactId>maven-compiler-plugin</artifactId> | ||
| <configuration> | ||
| <!-- | ||
| Below tests fail with hadoop 2 due to test dependency versioning issues. Hence, skip LDAP tests for Hadoop 2, | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This approach is clever but quite unexpected for the average dev and thus a bit scary to me.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| Retriggered build! | 
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
…ation mechanism (Web UI only)" to branch-2 Co-authored-by: Yash Dodeja <[email protected]>
… class SchemaLdifExtractor in ApacheDS 2.0.0.AM26. The issue arises from an overridden ApacheDS version incompatible with Hadoop 2. To resolve, skip LDAP tests for Hadoop 2 and apply dependency changes only for Hadoop 3.
| Rebased PR! | 
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| Failures are not related! | 
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| Failures are not related! | 
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
      
        
              This comment has been minimized.
        
        
      
    
  This comment has been minimized.
| 🎊 +1 overall 
 
 This message was automatically generated. | 
| 🎊 +1 overall 
 
 This message was automatically generated. | 
| 🎊 +1 overall 
 
 This message was automatically generated. | 
| 🎊 +1 overall 
 
 This message was automatically generated. | 
| Green build ! | 
…ation mechanism (Web UI only)" to branch-2 (apache#5144) (apache#5680) * Fix failure in pre-commit checks for Yetus JDK8 Hadoop2 due to missing class SchemaLdifExtractor in ApacheDS 2.0.0.AM26. The issue arises from an overridden ApacheDS version incompatible with Hadoop 2. To resolve, skip LDAP tests for Hadoop 2 and apply dependency changes only for Hadoop 3. Co-authored-by: Yash Dodeja <[email protected]> Signed-off-by: Nick Dimiduk <[email protected]>
…ation mechanism (Web UI only)" to branch-2
Co-authored-by: Yash Dodeja [email protected]