cbickel commented on issue #3104: Add TID stride configuration URL: https://github.com/apache/incubator-openwhisk/pull/3104#issuecomment-357172861 @rabbah Regarding your last comment: `If the point of the original counter change to use instances was to make non overlapping ids that worked because it separated even and odd numbers. But with more instances the numbers will overlap anyway.` Not only odd and even numbers are used. The total number of controllers is also part of the tid generation. E.g for 3 controllers: controller0 will have the following tids: 3, 6, 9, ... controller1: 4, 7, 10, ... controller2: 5, 8, 11, ... ```scala // Initial value: amountOfControllers + instanceOfThisController private lazy val cnt = new AtomicInteger(numberOfInstances + instanceOrdinal) def transid(): TransactionId = { // add amountOfControllers to the last tid TransactionId(cnt.addAndGet(numberOfInstances)) } ```
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
