yaojiejia

Hello! How complex should this unit's evaluation be? For example, do we need to consider cases like evaluate('ceil( 10.42ft / 2m )') and will ceil to a selected unit? Or...

> Sorry for the delay in reviewing. I have been traveling and did not see this PR. > > It seems given that `EXTRACT` has a special syntax, and the...

Just did! I added two options in ```PostgresOption``` which are ```--test-tablespaces``` and ```--tablespace-path```. For ```--test-tablespaces```, the default is disabled for all the other OS except linux because I don't have...

Hi @mrigger , just updated the PR and can you lmk any additional changes that should be made? Thanks

Hmm strange! I ran all the tests locally and they seem to be fine, will investigate further  

> There is another PR that already addresses this: #1171 > > Did you already continue on this PR? Otherwise, I'd be inclined to focus on reviewing and merging the...

> I think the code is currently used only by the Query Plan Guidance (QPG) component. If we execute `ANALYZE` for it, performance would drop. I guess we could have...

> @yaojiejia, I see you have been working on multiple other PRs. Do you think it would be possible to focus on the open ones, so we can merge those...

> We could add an enum (see > > https://github.com/sqlancer/sqlancer/blob/49c735829617aef9f4dc351b5844a32cbeb6b441/src/sqlancer/postgres/PostgresProvider.java#L129 > ). In this enum, we can specify a newly-created method in the explain generator that can generate the full...

No worries!