Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for deleting items #8

Closed
carlosmiranda opened this issue Mar 12, 2014 · 33 comments
Closed

Support for deleting items #8

carlosmiranda opened this issue Mar 12, 2014 · 33 comments

Comments

@carlosmiranda
Copy link

We're looking to migrate the S3Auth project to jcabi-dynamo (yegor256/s3auth#90). In jcabi-dynamo, there doesn't appear to be any support for deleting items:

        final AmazonDynamoDB amazon = this.client.get();
        amazon.deleteItem(
            new DeleteItemRequest(
                this.table,
                new ImmutableMap.Builder<String, AttributeValue>()
                    .put(DefaultDynamo.NAME, new AttributeValue(domain.name()))
                    .build()
            )
        );

What I end up doing is that I get the AmazonDynamoDB instance from Region. Can we add support for it in jcabi-dynamo?

@yegor256
Copy link
Member

You can delete an item through frame().iterator().remove(), but I agree that it's better to have a possibility to delete an individual item through Item.remove()

@yegor256
Copy link
Member

thanks for reporting

@yegor256
Copy link
Member

@dmarkov please pay for it

@carlosmiranda
Copy link
Author

@dmarkov This hasn't been paid for yet.

@yegor256
Copy link
Member

@dmarkov hey, please pay for this report

@dmarkov
Copy link

dmarkov commented Mar 19, 2014

@dmarkov hey, please pay for this report

@yegor256 sorry, but @carlosmiranda is not a member of the team (or I've got wrong information)

@yegor256
Copy link
Member

@dmarkov my bad, now he is, please pay for the report

@dmarkov
Copy link

dmarkov commented Mar 19, 2014

@dmarkov my bad, now he is, please pay for the report

@yegor256 sorry, but @carlosmiranda is not a member of the team (or I've got wrong information)

@carlosmiranda
Copy link
Author

@yegor256 , @dmarkov , will this report be paid for? I'm confused...

@yegor256
Copy link
Member

@dmarkov try again, please pay for this report

@dmarkov
Copy link

dmarkov commented Mar 25, 2014

@dmarkov try again, please pay for this report

@yegor256 sorry, but @carlosmiranda is not a member of the team (or I've got wrong information)

@carlosmiranda
Copy link
Author

@dmarkov , @yegor256 , any updates on the payment for this report?

@yegor256
Copy link
Member

@dmarkov please try again and pay for this report

@carlosmiranda
Copy link
Author

@dmarkov , @yegor256 , when will this report will be paid for?

@dmarkov
Copy link

dmarkov commented Apr 16, 2014

@dmarkov please try again and pay for this report

@yegor256 yes, it's done now @carlosmiranda thank you for this report, I've added 15 mins to your account in payment 38483766

@dmarkov
Copy link

dmarkov commented Jul 21, 2014

@hariso please help, the task is yours

@dmarkov
Copy link

dmarkov commented Sep 5, 2014

@hariso you've been working with this task for too long. I'm sorry, but I'll assign someone else. Please stop working with it. See our no obligations principle

@dmarkov dmarkov removed the @hariso label Sep 5, 2014
@dmarkov
Copy link

dmarkov commented Oct 24, 2014

@antacerod this ticket is yours now, please proceed, and keep in mind this. Any technical questions you should ask right here

@dmarkov
Copy link

dmarkov commented Oct 24, 2014

@antacerod The cost of this task is 30 mins (this is exactly how much will be paid, not less not more), when the task is done

@dmarkov
Copy link

dmarkov commented Jan 1, 2015

@antacerod please stop working with it, somebody else will pick it up, thanks

@dmarkov
Copy link

dmarkov commented Jan 1, 2015

@pandreev this ticket is yours now, please proceed, and keep in mind this. Any technical questions you should ask right here

Task's budget is 30 mins (see this for explanation)

pandreev added a commit to pandreev/jcabi-dynamo that referenced this issue Jan 2, 2015
pandreev added a commit to pandreev/jcabi-dynamo that referenced this issue Jan 6, 2015
@dmarkov
Copy link

dmarkov commented Jan 14, 2015

@pandreev the task is your hands for the last 13 days.. keep in mind that if it's not closed in the next 24 hours, it will be re-assigned to someone else, see No Obligations principle; -30 added to your rating, current score is: -90

@dmarkov
Copy link

dmarkov commented Jan 16, 2015

@pandreev you've been working with this task for too long. I'm sorry, but I'll assign someone else. Please stop working with it. See our no obligations principle. -60 to your rating, your total score is -210

@dmarkov
Copy link

dmarkov commented Jan 20, 2015

@nhekfqn This task is yours, please go ahead keeping in mind this. If any questions, don't hesitate to ask right here; The budget of this issue is 30 mins, which is exactly how much will be paid when the task is done (see this for explanation)

This was referenced Jan 20, 2015
@nhekfqn
Copy link
Contributor

nhekfqn commented Jan 25, 2015

@dmarkov PR #29 has been submitted.

Two new tickets submitted (#25 #26) in order to implement new method Table.delete in other classes implementing Table. Puzzles for those tickets added in code.

@nhekfqn
Copy link
Contributor

nhekfqn commented Jan 25, 2015

@dmarkov I also mentioned Checkstyle checks cannot be suppressed with @SuppressWarnings annotation, so I submitted another ticket to qulice yegor256/qulice#392

@nhekfqn
Copy link
Contributor

nhekfqn commented Jan 28, 2015

@dmarkov Related PR has been merged.

@nhekfqn
Copy link
Contributor

nhekfqn commented Jan 28, 2015

@carlosmiranda Could you close the ticket please?

@carlosmiranda
Copy link
Author

@nhekfqn Looks good, thanks! Closing...

@nhekfqn
Copy link
Contributor

nhekfqn commented Jan 29, 2015

@dmarkov The ticket has been closed.

@dmarkov
Copy link

dmarkov commented Feb 2, 2015

@nhekfqn 30 mins added to your account (payment number 51004274), many thanks for your contribution!; +30 added to your rating, current score is: +135

@dmarkov
Copy link

dmarkov commented Feb 6, 2015

2 puzzles were created here: 8-abdd4627, 8-81cf3845

@dmarkov
Copy link

dmarkov commented Sep 2, 2015

@carlosmiranda the last puzzle 8-abdd4627/#35 originated from here solved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants