Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-28 Thread via GitHub


syhily commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2081379636

   OK, feel free to reopen this PR if you need.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-27 Thread via GitHub


wenbingshen closed pull request #91: [FLINK-35196] [Connector / Pulsar] Fix 
bouncycastle class not found
URL: https://github.com/apache/flink-connector-pulsar/pull/91


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-27 Thread via GitHub


wenbingshen commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2081358915

   @syhily ignore this problem. Thanks.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-26 Thread via GitHub


syhily commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2079354340

   > > The pulsar-client-all should be upgrade to the 3.0.4 in the mean time.
   > 
   > @syhily After `pulsar-client-all` was updated to 3.0.4, the unit test 
failed. I took a brief look and found that `publishTimeStopCursor` seemed to be 
connected to the consumer and then seek position, so consumer will be 
disconnect.Then after reconnecting the consumer, an exception of `Exclusive 
consumer is already connected` occurred. I think this requires more detailed 
investigation shows that this may be due to some changes in the internal 
implementation of pulsar-client-all due to version upgrades, and 
troubleshooting is not that easy.
   > 
   > So I think it is worth using a separate PR to upgrade pulsar-client-all to 
version 3.0.4 and solve the problems caused by the related upgrade.
   > 
   > This PR only solves the problem that the table api cannot find the 
bouncycastle class.
   > 
   > Related error logs: 
[apache/flink-connector-pulsar/actions/runs/8832359197/job/24249517876?pr=91](https://github.com/apache/flink-connector-pulsar/actions/runs/8832359197/job/24249517876?pr=91)
   > 
   > PTAL. Thanks.
   
   Thanks for your works. I think we can divide the work into two steps. Bump 
pulsar-client-all do require a lot of works. But since the pulsar-client-all 
depends on the legacy 15on. I think we can exclude these dependencies first in 
this PR and use 18on.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-25 Thread via GitHub


wenbingshen commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2078583839

   > The pulsar-client-all should be upgrade to the 3.0.4 in the mean time.
   
   @syhily 


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-25 Thread via GitHub


wenbingshen closed pull request #91: [FLINK-35196] [Connector / Pulsar] Fix 
bouncycastle class not found
URL: https://github.com/apache/flink-connector-pulsar/pull/91


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-25 Thread via GitHub


wenbingshen commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2076975181

   > The pulsar-client-all should be upgrade to the 3.0.4 in the mean time.
   
   @syhily sure. I have addressed.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-24 Thread via GitHub


syhily commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2075204586

   The pulsar-client-all should be upgrade to the 3.0.4 in the mean time.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-22 Thread via GitHub


wenbingshen commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2071359293

   > Thanks for your commit. I think `jdk15on` should be dropped by in favor of 
the `jdk18on`.
   
   @syhily Thanks review. I have upgraded `jdk15on` to `jdk18on`. PTAL.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-22 Thread via GitHub


syhily commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2069874438

   Thanks for your commit. I think `jdk15on` should be dropped by in favor of 
the `jdk18on`.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-22 Thread via GitHub


wenbingshen commented on PR #91:
URL: 
https://github.com/apache/flink-connector-pulsar/pull/91#issuecomment-2068796453

   @syhily @snuyanzin @tsreaper PTAL. Thanks.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [FLINK-35196] [Connector / Pulsar] Fix bouncycastle class not found [flink-connector-pulsar]

2024-04-22 Thread via GitHub


wenbingshen closed pull request #91: [FLINK-35196] [Connector / Pulsar] Fix 
bouncycastle class not found
URL: https://github.com/apache/flink-connector-pulsar/pull/91


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org