default db_schema to 'public'

[shaibruhis]

As stated on Postgres's website, the default schema name is "public".

In the previous sections we created tables without specifying any schema names. By default such tables (and other objects) are automatically put into a schema named “public”.

[shaibruhis]

@nossrannug not sure if you saw this or not :)

[nossrannug]

Hey @shaibruhis , things for the PR. I'll take a look at this tomorrow. I'm hoping to release a new version of feast-postgres then to support feast 0.19 and I might include this. Giving it some thought because it would be a breaking change for everyone that is using the default value

[shaibruhis]

I do think it makes more sense to have it default to public but totally understand the concern for backward compatability!

Hoping it makes it in but no worries if not :)

[shaibruhis]

@nossrannug, curious if you've had any more thoughts here or if I should close it out?

[nossrannug]

I won't be making any more changes to this repo since I'm working on moving the postgres functionality to Feast feast-dev/feast#2401

You can add a comment to the PR and we can get feedback from the people at Feast on what they think :)

[shaibruhis]

Awesome! Congrats on merging it and looking forward to using it in Feast :)