[GitHub] [pulsar] codelipenghui commented on pull request #8422: [PIP-71][SQL]Pulsar SQL migrate SchemaHandle to presto decoder

[GitBox <gi...@apache.org>]

codelipenghui commented on pull request #8422:
URL: https://github.com/apache/pulsar/pull/8422#issuecomment-770232003


   @hnail @gaoran10 Could you please help take a look the failed test? 
   ```
   Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project pulsar-presto-connector-original: Compilation failure
   Error:  /Users/runner/work/pulsar/pulsar/pulsar-sql/presto-pulsar/src/test/java/org/apache/pulsar/sql/presto/TestPulsarRecordCursor.java:[352,40] method asyncReadEntries in class org.apache.bookkeeper.mledger.impl.ManagedCursorImpl cannot be applied to given types;
   Error:    required: int,org.apache.bookkeeper.mledger.AsyncCallbacks.ReadEntriesCallback,java.lang.Object,org.apache.bookkeeper.mledger.impl.PositionImpl
   Error:    found: int,java.lang.Object,java.lang.Object
   Error:    reason: actual and formal argument lists differ in length
   ```
   The interesting thing this why other tests are passed. Might be related to the other tests only run the install for profile `core-modules` ?


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