[GitHub] [geode] sabbeyPivotal commented on pull request #5416: GEODE-8382: Run Redis tests against Redis API for Geode

2020-08-03 Thread GitBox 


sabbeyPivotal commented on pull request #5416:
URL: https://github.com/apache/geode/pull/5416#issuecomment-668168119


   @smgoller @rhoughton-pivot @onichols-pivotal @dickcav 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [geode] sabbeyPivotal commented on pull request #5416: GEODE-8382: Run Redis tests against Redis API for Geode

2020-07-31 Thread GitBox 


sabbeyPivotal commented on pull request #5416:
URL: https://github.com/apache/geode/pull/5416#issuecomment-667115984


   > The pipeline linked is not publicly viewable, so it is harder to assert 
that it is indeed working
   
   Good point, Mike.  The pipeline was tested locally, feel free to deploy a 
local version to make sure it works.
   DUnit failure is unrelated to this change.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [geode] sabbeyPivotal commented on pull request #5416: GEODE-8382: Run Redis tests against Redis API for Geode

2020-07-30 Thread GitBox 


sabbeyPivotal commented on pull request #5416:
URL: https://github.com/apache/geode/pull/5416#issuecomment-02055


   Tested new pipeline job here: 
https://concourse.gemfire-ci.info/teams/developer/pipelines/sabbeypivotal-pipelining-pr



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org