Conversation
|
would love to see this merged, is anything left to do? can I help? |
Config option to supply extra parameters to the JDBC string for advanced cases. Example: Useful for connecting to Postgres behind pgbouncer when set to ?prepareThreshold=0 Signed-off-by: PseudoResonance <kaio11604@gmail.com>
Disables index hinting for Postgres Adds subquery aliases when creating unions Signed-off-by: PseudoResonance <kaio11604@gmail.com>
Signed-off-by: PseudoResonance <kaio11604@gmail.com>
|
This is not ready to merge yet, but I've made a few fixes on my own fork. I wasn't able to find any more issues, but if anyone would like to try it out on a test server and see if there are any errors I would appreciate it. Definitely do NOT use this on a production server though. https://github.com/PseudoResonance/CoreProtect/ |
Fix Postgres lookups/JDBC parameter config option
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
Why is there a stale bot for PRs? |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
Still not stale |
|
should I setup a bot that comments every day or week something random? |
|
Hi, I want to add another RDBMS other than PostgreSQL to this project, I think with this code it is easy to add other RDBMS, will this pull request be added in the future? |
|
Well right now this PR isn't updated to the latest codebase so that would be a blocker initially |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
could we please merge this now? |
If the PR is updated to the latest code then intelli could actually consider it |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
Please merge |
1 similar comment
|
Please merge |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
please merge |
|
9 participants
Fixes #64