[ https://issues.apache.org/jira/browse/GEODE-9324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17492733#comment-17492733 ]
ASF GitHub Bot commented on GEODE-9324: --------------------------------------- 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: notifications-unsubscr...@geode.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Remove ACE_Task references > -------------------------- > > Key: GEODE-9324 > URL: https://issues.apache.org/jira/browse/GEODE-9324 > Project: Geode > Issue Type: Improvement > Components: native client > Reporter: Mario Salazar de Torres > Assignee: Mario Salazar de Torres > Priority: Major > Labels: obliterate-ace, pull-request-available > > *AS A* native client contributor > *I WANT TO* remove all remaining references to ACE_Task > *SO THAT* eventually we can get rid of ACE library > -- This message was sent by Atlassian Jira (v8.20.1#820001)