-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Issue #3290 Pulsar IO connector for Hbase sink #3368
Conversation
rerun java tests |
run Integration Tests error: |
run Integration Tests |
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.
can you also add a documentation page? also if it is possible, it would be great to add an integration test.
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/HbaseAbstractConfig.java
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/HbaseAbstractConfig.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/HbaseAbstractConfig.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseSinkConfig.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseGenericRecordSink.java
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/HbaseAbstractConfig.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
@ambition119 did you push new changes? I saw you resolved the review conversations without pushing new changes. I am guessing you made local changes but didn't push to remote. |
sorry, I forgot to submit the code. |
run cpp Tests for error: run Integration Tests for error: |
pulsar-client/src/main/java/org/apache/pulsar/client/impl/schema/BooleanSchema.java
Outdated
Show resolved
Hide resolved
pulsar-client/src/main/java/org/apache/pulsar/client/impl/schema/BooleanSchema.java
Outdated
Show resolved
Hide resolved
pulsar-client/src/main/java/org/apache/pulsar/client/impl/schema/BooleanSchema.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
run Integration Tests Pulsar SQL :: Pulsar Presto Distribution ........... FAILURE [ 18.556 s] |
run Integration Tests org.testcontainers.containers.ContainerLaunchException: Container startup failed |
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.
@ambition119 can you create a github issue for adding an integration test for hbase connectors?
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Show resolved
Hide resolved
pulsar-io/hbase/src/main/java/org/apache/pulsar/io/hbase/sink/HbaseAbstractSink.java
Outdated
Show resolved
Hide resolved
run java8 tests |
hbase integration test code also ok, then create new PR and review. |
merge into myself
run java8 Tests |
Well done! @ambition119 |
Supporting the ability to write data to HBase sink #3290