amrishlal commented on pull request #7174: URL: https://github.com/apache/pinot/pull/7174#issuecomment-897015742
@mcvsubbu @jackjlli Based on our offline discussion, I modified code to use a `Properties` object to store table name and setting it under lock to ensure that there are no thread synchronization issues if we end up in situation where multiple threads are running the same task. Also confirming that task will block only if another thread is running the same task at the same time, but different tasks won't block each other). Also, now we only specify one table in the API along with its type (OFFLINE|ONLINE) since this appears to be the pattern that is being used for other API calls too. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
