delta icon indicating copy to clipboard operation
delta copied to clipboard

DELETE should return the number of deleted rows

Open edmondop opened this issue 3 years ago • 3 comments

Description

Resolves #1222

How was this patch tested?

The SQL test suite was extended

Does this PR introduce any user-facing changes?

The returned DataFrame from a delete should not be empty, but it will contain a single row

edmondop avatar Jun 29 '22 23:06 edmondop

Can you fix the title to be "DELETE should return the number of deleted rows"?

jaceklaskowski avatar Jun 30 '22 05:06 jaceklaskowski

Hey @edmondo1984 we'd love to resolve #1222 are you still interested in working on this PR?

allisonport-db avatar Aug 04 '22 17:08 allisonport-db

I've submitted a PR here: https://github.com/delta-io/delta/pull/1328

@edmondo1984 want to take a look?

scottsand-db avatar Aug 10 '22 22:08 scottsand-db