Re: [PR] [hotfix] Add jakarta.activation required after changes in Flink main repo [flink-connector-jdbc]

2024-01-26 Thread via GitHub
snuyanzin closed pull request #93: [hotfix] Add jakarta.activation required after changes in Flink main repo URL: https://github.com/apache/flink-connector-jdbc/pull/93 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [hotfix] Add jakarta.activation required after changes in Flink main repo [flink-connector-jdbc]

2024-01-26 Thread via GitHub
snuyanzin commented on PR #93: URL: https://github.com/apache/flink-connector-jdbc/pull/93#issuecomment-1912104156 closing since the issue was fixed in upstream -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [hotfix] Add jakarta.activation required after changes in Flink main repo [flink-connector-jdbc]

2024-01-19 Thread via GitHub
snuyanzin commented on PR #93: URL: https://github.com/apache/flink-connector-jdbc/pull/93#issuecomment-1899938305 Thanks, for having a look however i'm now wondering whether we can just wait and close it later since it's going to be fixed in Flink itself

Re: [PR] [hotfix] Add jakarta.activation required after changes in Flink main repo [flink-connector-jdbc]

2024-01-18 Thread via GitHub
snuyanzin commented on PR #93: URL: https://github.com/apache/flink-connector-jdbc/pull/93#issuecomment-1899894520 @BlackPigHe i didn't get your comment have you seen the changes proposed within this PR? -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [hotfix] Add jakarta.activation required after changes in Flink main repo [flink-connector-jdbc]

2024-01-18 Thread via GitHub
BlackPigHe commented on PR #93: URL: https://github.com/apache/flink-connector-jdbc/pull/93#issuecomment-1899867428 I also found this problem, and found that the work flow kept failing when mr -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [hotfix] Add jakarta.activation required after changes in Flink main repo [flink-connector-jdbc]

2024-01-18 Thread via GitHub
BlackPigHe commented on PR #93: URL: https://github.com/apache/flink-connector-jdbc/pull/93#issuecomment-1899866113 javax.activation activation 1.1.1 test

[PR] [hotfix] Add jakarta.activation required after changes in Flink main repo [flink-connector-jdbc]

2024-01-18 Thread via GitHub
snuyanzin opened a new pull request, #93: URL: https://github.com/apache/flink-connector-jdbc/pull/93 The reason is this line