search

adjust / parquet_fdw

Parquet foreign data wrapper for PostgreSQL
PostgreSQL License
351 stars 38 forks source link

Refactor README.md for PGXN #64

Closed mkgrgis closed 8 months ago

mkgrgis commented 1 year ago

For unifying with https://github.com/pgspider FDW documentation template and by https://github.com/ibarwick/firebird_fdw/blob/master/README.md example of advanced FDW documentation.

mkgrgis commented 8 months ago

Ping @za-arthur , any review of this PR for PostgreSQL extension network - FDW page ?

za-arthur commented 8 months ago

Sorry, I don't get why these changes are necessary.

mkgrgis commented 8 months ago

@za-arthur, new README.md follows more clear and unified structure which is used by READMEs in most of other PostgreSQL FDWs. This README structure is de facto better practice and you can ensure similar structure is yet used in https://github.com/pgspider/sqlite_fdw , https://github.com/pgspider/influxdb_fdw , https://github.com/pgspider/dynamodb_fdw , https://github.com/laurenz/oracle_fdw/ , https://github.com/ibarwick/firebird_fdw etc.

za-arthur commented 8 months ago

parquet_fdw is a separate project and doesn't related to those projects. It might have its own README which might be different from READMEs of other projects.

mkgrgis commented 8 months ago

It might have its own README which might be different from READMEs of other projects.

Yes, of course. This was PR for less differences between forks from pgspider project which also use this FDW as a part.