mongodb / mongo-jdbc-driver

JDBC Driver for MongoDB Atlas SQL interface
Apache License 2.0
37 stars 33 forks source link

SQL-513: Implement MongoSQLResultSetMetaData #47

Closed pmeredit closed 2 years ago

pmeredit commented 2 years ago

Ultimately, I needed to use an ExtendedBsonType so that we could easily represent dynamically types/unknown type column

EDIT: We moved away from this approach, now everything is in a MongoColumnInfo class

pmeredit commented 2 years ago

@mattChiaravalloti @rychipman went ahead and rebased on master since no one has reviewed this yet!

pmeredit commented 2 years ago

@rychipman @mattChiaravalloti ok, the massive refactor is finished.

pmeredit commented 2 years ago

@rychipman this has been rebased on master, MongoJsonSchema was moved into a subtask

pmeredit commented 2 years ago

@rychipman I really like the suggestion on boolean required. I actually simplified the logic quite a bit based on that.