[GitHub] storm pull request: Storm-818: storm-eventhubs configuration impro...

[shanyu]

Github user shanyu commented on the pull request:

    https://github.com/apache/storm/pull/552#issuecomment-107665673
  
    Hi Taylor,
    
    One of the dependencies (Apache Qpid) requires Java 1.7 and above, 
therefore you have to use Java 1.7 to build and run unit tests for 
storm-eventhubs. You’ll see those exceptions when using Java 1.6 to build the 
project.
    
    What is the Java version used on the build server? Is there a requirement 
for Java versions for Storm? I couldn’t find any info online.
    
    The Qpid library is a must for EventHubSpout to work properly. So if we 
cannot change Java version requirement for Storm, I’ll have to disable a few 
unit tests to let it pass on Java 1.6.
    
    Thanks,
    Shanyu
    
    From: P. Taylor Goetz [mailto:notificati...@github.com]
    Sent: Monday, June 1, 2015 11:01 AM
    To: apache/storm
    Cc: Shanyu Zhao
    Subject: Re: [storm] Storm-818: storm-eventhubs configuration improvement 
and refactoring (#552)
    
    
    @shanyu<https://github.com/shanyu>, I'm getting test failures with the 
latest code:
    
    Tests in error:
    
      testSpoutCheckpoint(org.apache.storm.eventhubs.spout.TestEventHubSpout): 
org/apache/qpid/amqp_1_0/type/messaging/Data : Unsupported major.minor version 
51.0
    
      testSpoutBasic(org.apache.storm.eventhubs.spout.TestEventHubSpout): 
org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPMCheckpointWithPendingAndResend(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPartitionManagerNoFail(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPartitionManagerResend(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPMCheckpointWithResend(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPMCheckpointWithNoPendingAndNoResend(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPartitionManagerMaxPendingMessages(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPMCheckpointWithPending(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testPartitionManagerEnqueueTimeFilter(org.apache.storm.eventhubs.spout.TestPartitionManager):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testEmitInSequence(org.apache.storm.eventhubs.trident.TestTransactionalTridentEmitter):
 org/apache/qpid/amqp_1_0/type/messaging/Data
    
      
testReEmit(org.apache.storm.eventhubs.trident.TestTransactionalTridentEmitter): 
org/apache/qpid/amqp_1_0/type/messaging/Data
    
    Can you take a look?
    
    —
    Reply to this email directly or view it on 
GitHub<https://github.com/apache/storm/pull/552#issuecomment-107655396>.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---