You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "laglangyue (via GitHub)" <gi...@apache.org> on 2024/03/23 07:23:16 UTC

[PR] upgrade mssql version to 12.6.1 [pekko-persistence-jdbc]

laglangyue opened a new pull request, #161:
URL: https://github.com/apache/pekko-persistence-jdbc/pull/161

   close #113 
   
   I search the changes in mssql release
   https://github.com/microsoft/mssql-jdbc/releases?q=encrypt&expanded=true
   look at this picture. 
   ![image](https://github.com/apache/pekko-projection/assets/35491928/3e693b8c-7de8-4fec-a239-da6a2e700634)
   
   ## How to solve this for our test
   (1) add `encrypt=true;trustServerCertificate=true"` to jdbc url
   (2) or add `encrypt=false` to jdbc url
   ## What can users do?
   users should ensure that the sqlserver enable tls,maybe need some config, so that client can connect to sqlserver with TLS


-- 
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: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org


Re: [PR] upgrade mssql version to 12.6.1 [pekko-persistence-jdbc]

Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning merged PR #161:
URL: https://github.com/apache/pekko-persistence-jdbc/pull/161


-- 
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: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org


Re: [PR] upgrade mssql version to 12.6.1 [pekko-persistence-jdbc]

Posted by "laglangyue (via GitHub)" <gi...@apache.org>.
laglangyue commented on PR #161:
URL: https://github.com/apache/pekko-persistence-jdbc/pull/161#issuecomment-2016393827

   ![image](https://github.com/apache/pekko-persistence-jdbc/assets/35491928/a9b42709-fc62-4216-bcc4-b9b85183c8b9)
   
   I have test it locally, it looks good


-- 
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: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org


Re: [PR] upgrade mssql version to 12.6.1 [pekko-persistence-jdbc]

Posted by "laglangyue (via GitHub)" <gi...@apache.org>.
laglangyue commented on code in PR #161:
URL: https://github.com/apache/pekko-persistence-jdbc/pull/161#discussion_r1536586377


##########
.gitignore:
##########
@@ -12,3 +12,4 @@ target
 *.code-workspace
 .bloop
 .metals
+dumps

Review Comment:
   when I test in local ,this folder generate, I think adding it to ignore  file would be better



-- 
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: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org


Re: [PR] upgrade mssql version to 12.6.1 [pekko-persistence-jdbc]

Posted by "laglangyue (via GitHub)" <gi...@apache.org>.
laglangyue commented on PR #161:
URL: https://github.com/apache/pekko-persistence-jdbc/pull/161#issuecomment-2016395153

   @pjfanning 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: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org