gaussianrecurrence commented on pull request #812: URL: https://github.com/apache/geode-native/pull/812#issuecomment-1040530307
> > Hi @pdxcodemonkey, among others interested I think this is ready to continue reviewing. Will be glad to hear any feedback :) > > Do you wish to address any of the variable names as mentioned, or no? If no just indicate here, and I'll approve/merge as is. TBH I thought I addressed them all on revision 1. However, even If there are more naming comments to be addressed, I'd say it's best to get this merged, since this PR has been long standing. And, also, I don't think it's worth to invest too much effort on the old ITs, just maintenance until we get the equivalent new ITs. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
