raboof commented on PR #7903:
URL: https://github.com/apache/geode/pull/7903#issuecomment-3227819334

   > Thank you for your feedback, @raboof. Could you please take a look at 
[9d736a9](https://github.com/apache/geode/commit/9d736a960ef03168a61b5a9d8aaa22c6e5c27daf)
 to see if it takes care of https://issues.apache.org/jira/browse/GEODE-10458 ?
   
   When you open that commit page, near the top you see "Commit 9d736a9, 
JinwooHwang-SAS authored 16 hours ago . x 14 / 16 . Verified". If you click the 
`x 14 / 16`, you see that `develop / integratonTest` still fails for that 
commit.
   
   If you click on 'Details' next to `develop / integratonTest`, the logs 
indeed reveal the `AnalyzeCoreSerializablesIntegrationTest / testSerializables` 
mentioned in https://issues.apache.org/jira/browse/GEODE-10458 is now fixed - 
congratulations! It would have been nice to mention GEODE-10458 in the #7915 
commit and pull request. I've updated the PR title for you.
   
   I see `develop / integratonTest` is now green on the main branch, awesome!
   
   It would be super cool if we could also make `develop / acceptanceTest` 
green, but if that's too complicated then perhaps we can skip it instead


-- 
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.

To unsubscribe, e-mail: notifications-unsubscr...@geode.apache.org

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

Reply via email to