You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/12/23 05:55:43 UTC
[GitHub] [pulsar] rdhabalia opened a new pull request #9036: [pulsar-io] upgrade debezium version
rdhabalia opened a new pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036
### Motivation
Upgeade debezium version for io connector.
----------------------------------------------------------------
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
[GitHub] [pulsar] rdhabalia commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
rdhabalia commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-750416858
/pulsarbot run-failure-checks
----------------------------------------------------------------
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
[GitHub] [pulsar] eolivelli commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-778172767
/pulsarbot run-failure-tests
----------------------------------------------------------------
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
[GitHub] [pulsar] codelipenghui commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
codelipenghui commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-750299811
/pulsarbot run-failure-checks
----------------------------------------------------------------
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
[GitHub] [pulsar] rdhabalia commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
rdhabalia commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-754109046
/pulsarbot run-failure-checks
----------------------------------------------------------------
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
[GitHub] [pulsar] eolivelli commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-749960107
Don't we have to update license files as well?
----------------------------------------------------------------
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
[GitHub] [pulsar] rdhabalia commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
rdhabalia commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-749978977
we don't have version defined into [license file](https://github.com/apache/pulsar/blob/master/pulsar-io/debezium/postgres/src/main/resources/META-INF/services/pulsar-io.yaml)
----------------------------------------------------------------
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
[GitHub] [pulsar] lxcid edited a comment on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
lxcid edited a comment on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-783486827
I hope I can help a bit, I'm diving the codes, and compiling. the process take a long time, meanwhile, I'm looking at the failing test and ~~it seems really weird that the test case is validating Avro instead of Json when the code obviously uses Json.~~
The stack trace is MySQL one. that explains why.
https://github.com/apache/pulsar/blob/16a2240a5af8402cf655049292d669378e694fa3/tests/integration/src/test/java/org/apache/pulsar/tests/integration/functions/PulsarFunctionsTest.java#L168-L171
I'll continue to dig to see if I can figure it out.
----------------------------------------------------------------
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
[GitHub] [pulsar] sijie commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-780264754
@rdhabalia Can you take a look at the integration failures? It might be still related to debezium.
----------------------------------------------------------------
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
[GitHub] [pulsar] eolivelli commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-778173231
@lxcid we have some issues on CI, if the failures are not related to the patch then we simply rerun the tests.
There is an ongoing effort to fix all flakes and also to reduce overall CI time for PR validation
----------------------------------------------------------------
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
[GitHub] [pulsar] lxcid commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
lxcid commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-779915866
/pulsarbot run-failure-tests
----------------------------------------------------------------
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
[GitHub] [pulsar] eolivelli commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-879747744
@rdhabalia this PR is superseded by the more recent #11204
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] lxcid commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
lxcid commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-778734860
Thanks @eolivelli for the reply! our assessment is still very rough and not reliable but we wanted to integrate pulsar for a while, and have been tracking the debezium part which is our blocker. finger cross in seeing it get landed! thanks so much!
----------------------------------------------------------------
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
[GitHub] [pulsar] sijie commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-751553727
/pulsarbot run-failure-checks
----------------------------------------------------------------
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
[GitHub] [pulsar] sijie commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-757614021
@rdhabalia it seems that debezium integration tests are still failing.
----------------------------------------------------------------
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
[GitHub] [pulsar] sijie commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-750447578
/pulsarbot run-failure-checks
----------------------------------------------------------------
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
[GitHub] [pulsar] lxcid commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
lxcid commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-783486827
I hope I can help a bit, I'm diving the codes, and compiling. the process take a long time, meanwhile, I'm looking at the failing test and it seems really weird that the test case is validating Avro instead of Json when the code obviously uses Json.
https://github.com/apache/pulsar/blob/16a2240a5af8402cf655049292d669378e694fa3/tests/integration/src/test/java/org/apache/pulsar/tests/integration/functions/PulsarFunctionsTest.java#L168-L171
I'll continue to dig to see if I can figure it out.
----------------------------------------------------------------
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
[GitHub] [pulsar] eolivelli closed pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
eolivelli closed pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] lxcid commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
lxcid commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-778150281
Is there anyway we can help to land this? i check the failure and see a few unrelated issue like rabbitmq and presto.
----------------------------------------------------------------
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
[GitHub] [pulsar] sijie commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-756580045
/pulsarbot run-failure-checks
----------------------------------------------------------------
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
[GitHub] [pulsar] sijie commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-756626412
@rdhabalia I think the debezium integration failed. Please take a look at those errors.
----------------------------------------------------------------
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
[GitHub] [pulsar] sijie commented on pull request #9036: [pulsar-io] upgrade debezium version
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #9036:
URL: https://github.com/apache/pulsar/pull/9036#issuecomment-756626412
@rdhabalia I think the debezium integration failed. Please take a look at those errors.
----------------------------------------------------------------
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