[GitHub] rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-09-12 Thread git
rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-328701240 pg3/1078 ? This is an automated message from the Apac

[GitHub] rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-09-11 Thread git
rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-328701240 pg3/1078 ? (because the last one has a build failure - unrelated).

[GitHub] rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-09-07 Thread git
rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-327980372 pg3/1068 ? This is an automated message from the Apac

[GitHub] rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-09-07 Thread git
rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-327980372 pg3/1067 ? This is an automated message from the Apac

[GitHub] rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-09-01 Thread git
rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-326559737 A side note that perhaps once the instances have unique, not ordinal, ids, the `tid` could be extended from `tid_NNN` to

[GitHub] rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-08-31 Thread git
rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-326356312 The patch here addressed a bug: the invoker tids would be an infinite sequence of 0s. The invoker however does not gener