-
Notifications
You must be signed in to change notification settings - Fork 19
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
Working sample for MySQL #30
Comments
Hi tailmax. How soon can you get it working with MySql? |
@Iulian94 I had no immediate plans of using MySql, as I don't use it anywhere. However currently provided SqlRepository can work with MySql. It will be a matter of providing correclty formed connection string (See PetaPoco documentation for that). However I sense there will be an issue in creating an initial schema for the tables, as SQL I used worked in SQL Server and I never tested it in MySql. But you can try an report back if there is an issue. |
@trailmax Thanks a lot for the quick response! I've tried it, and it does not work. I've downloaded the binaries, will try to figure out what is wrong and come back to you. at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor) at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, DbConnectionPool pool, String accessToken, Boolean applyTransientFaultHandling) |
@Iulian94 What is the actual message of the exception? How does your connection string look like? |
@trailmax
which works. Small problem though.. MySql doesn't have the notion of Schemas, ergo, it crashes. I resolved this by taking the binaries and changing the const properties, but, it's not ok, as I'm not using the Nuget anymore. |
@Iulian94 problem with renaming tables - it'll break all the existing solutions for people who are already using this package. Including my solutions. So it'll have to be a separate implementation of the repository and other DB-related structure for MySql. Since you already have the code working, would you like to share it via a Pull Request? |
@trailmax I've made the pull request #37.
The only problem that I had is that SagaData is internal and I can't inject the IQueryWrapper, because the constructor in public. |
@trailmax Can you please tell me whether it would be possible to carry this out until tomorrow, and fast forward the deployment of the package? |
@Iulian94 I'm snowed under by requests for work in paid projects. And that unfortunately takes priority. So unlikely there will be a relese for tomorrow. |
@trailmax Ok, thanks.. |
@Iulian94 OK, you are setting my priorities, giving me a deadline and you are getting paid for your project. Should I ask to be paid for my time as well? |
@trailmax I didn't mean it that way.. I just wanted to let you know that I also have a time limit on this, and that's why I was trying to push it that hard. Sorry if I was stressing you out so much with this. |
@trailmax Thanks for doing the code review for the pull request. |
@trailmax I've added the last pull request #38. |
No description provided.
The text was updated successfully, but these errors were encountered: