java.jdbc

creating indices - patch?

Details

  • Type: Enhancement Enhancement
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

java.jdbc currently has no support for creating and deleting indices. I wrote create-index-ddl for my own project to handle this; I could work that into a patch if that would be helpful. I know you're re-working the global DB connection right now, so if you'd like a patch, but would like me to wait until the code has settled down, that's fine, too.

Activity

Hide
Sean Corfield added a comment -

Thanx Matt. Definitely interested. DDL is an area the library is weak in and I'm thinking of adding clojure.java.jdbc.ddl as a namespace dedicated to DDL-generating functions, if you'd like to kick that off?

Show
Sean Corfield added a comment - Thanx Matt. Definitely interested. DDL is an area the library is weak in and I'm thinking of adding clojure.java.jdbc.ddl as a namespace dedicated to DDL-generating functions, if you'd like to kick that off?
Hide
Matt Oquist added a comment -

Here's a patch to add ddl/create-index and ddl/drop-index. I didn't add clojure.java.jdbc/create-index (and drop-) since clojure.java.jdbc/create-table is deprecated.

Show
Matt Oquist added a comment - Here's a patch to add ddl/create-index and ddl/drop-index. I didn't add clojure.java.jdbc/create-index (and drop-) since clojure.java.jdbc/create-table is deprecated.
Hide
Sean Corfield added a comment -

Thanx. The patch is not in the correct format (it's just a diff, not a patch) - can you redo it?

Show
Sean Corfield added a comment - Thanx. The patch is not in the correct format (it's just a diff, not a patch) - can you redo it?
Hide
Matt Oquist added a comment -

NP; sorry about that. Will this do?

Show
Matt Oquist added a comment - NP; sorry about that. Will this do?
Hide
Sean Corfield added a comment -

Perfect thanx!

Show
Sean Corfield added a comment - Perfect thanx!
Hide
Sean Corfield added a comment -

Patch merged. Documentation updated.

Show
Sean Corfield added a comment - Patch merged. Documentation updated.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: