Support partial index

[kyoto7250]

PR Info

close #396
We supports partial index by this PR.

Adds

• support partial index.

[ikrivosheev]

@kyoto7250 thank you! LGTM!

[billy1624]

Hey @kyoto7250, thanks again for contributing!!

For index statement, it doesn't support value binding. So, instead of writing a placeholder to the SQL - ? for MySQL and $N for PostgreSQL. It should just write the value in string. This QueryBuilder::value_to_string would come in handy.

Suggested change

	/// assert_eq!(
	/// index.to_string(PostgresQueryBuilder),
	/// r#"CREATE INDEX "idx-glyph-aspect" ON "glyph" ("aspect" (64) ASC) WHERE "glyph"."aspect" IN ($1, $2)"#
	/// );
	/// assert_eq!(
	/// index.to_string(SqliteQueryBuilder),
	/// r#"CREATE INDEX "idx-glyph-aspect" ON "glyph" ("aspect" ASC) WHERE "glyph"."aspect" IN (?, ?)"#
	/// );
	/// assert_eq!(
	/// index.to_string(PostgresQueryBuilder),
	/// r#"CREATE INDEX "idx-glyph-aspect" ON "glyph" ("aspect" (64) ASC) WHERE "glyph"."aspect" IN (3, 4)"#
	/// );
	/// assert_eq!(
	/// index.to_string(SqliteQueryBuilder),
	/// r#"CREATE INDEX "idx-glyph-aspect" ON "glyph" ("aspect" ASC) WHERE "glyph"."aspect" IN (3, 4)"#
	/// );

[billy1624]

Hey @kyoto7250, got any updates on this? :)

[ikrivosheev]

@kyoto7250 hello! Any updates?)

[ikrivosheev]

@kyoto7250 please, resolve conflict)

[kyoto7250]

Hello.

I missed that this PR exists, so I have not responded to @billy1624 's comment .

If you are not in a hurry, could you please wait a little longer?

If you are in a hurry, you can close this PR and create another one.

[ikrivosheev]

Hello.

I missed that this PR exists, so I have not responded to @billy1624 's comment .

If you are not in a hurry, could you please wait a little longer?

If you are in a hurry, you can close this PR and create another one.

@kyoto7250 take you time!

[kyoto7250]

I couldn't fix the conflicts cleanly, so I created #478.

This PR is old and I will close it.