Re: Data ingestion with predefined buckets

2020-04-11 Thread Udo Kohlmeyer
Hi there Steve, Firstly, you are correct, the pattern you are describing is not recommended and possibly not even correctly supported. I've seen many implementations of Geode systems and none of them ever needed to do what you are intending to do. Seems like you are will to go through A LOT of

Help with Concourse-CI redis integration test failing

2020-04-11 Thread Alberto Gomez
Hi, I am seeing the following integration test case fail in the Concourse-CI when executed on my pull request (https://github.com/apache/geode/pull/4928): org.apache.geode.redis.SetsIntegrationTest > testSRem_should_notRemoveMembersOfSetNotSpecified FAILED Interestingly, if I run the test loca

Re: Help with Concourse-CI redis integration test failing

2020-04-11 Thread Owen Nichols
Hi Alberto, it looks like Integration tests have been broken since PR#4937 was merged at the end of the day Friday. However this PR had passed all its PR checks (as of 5 hours before it was merged) so I suspect this was just an unexpected interaction with the PR#4912 that came in just 3 hours b

Re: Help with Concourse-CI redis integration test failing

2020-04-11 Thread Alberto Gomez
Thanks Owen. It seems you already initiated the revert. Best regards, -Alberto From: Owen Nichols Sent: Saturday, April 11, 2020 8:04 PM To: dev@geode.apache.org Subject: Re: Help with Concourse-CI redis integration test failing Hi Alberto, it looks like Integ

Re: Help with Concourse-CI redis integration test failing

2020-04-11 Thread Owen Nichols
Looks like GitHub creates a branch for the revert as soon as someone clicks Revert (I didn’t mean to, I just wanted to see what the next screen looks like, I didn’t click Create Pull Request). If I create the pull request, then we need to find a 3rd person to approve it, so hoping you can still

Re: Help with Concourse-CI redis integration test failing

2020-04-11 Thread Alberto Gomez
I clicked on the Create Pull Request button but got the following error: Pull request creation failed. Validation failed: You can't perform that action at this time. -Alberto From: Owen Nichols Sent: Saturday, April 11, 2020 8:35 PM To: dev@geode.apache.org Su

Re: Help with Concourse-CI redis integration test failing

2020-04-11 Thread Alberto Gomez
Owen, I managed to create the revert pull request original pull request page. https://github.com/apache/geode/pull/4947 [https://avatars3.githubusercontent.com/u/47359?s=400&v=4] Revert "GEODE-7978: Improve tests for Redis Module SREM Command" by albertog

Re: Help with Concourse-CI redis integration test failing

2020-04-11 Thread Owen Nichols
Merged the revert PR to develop. John or Jens, please have another try (create a new branch from latest develop then revert the revert commit) Alberto, and anyone else with a PR affected by this, just push an empty commit or another commit to your PR (or merge or rebase latest develop, if you p