Process defenition id generator

[hammer]

Guys, it's not funny in BpmnDeployer.class
<blockcode>
        String nextId = idGenerator.getNextId();
        String processDefinitionId = processDefinition.getKey()
          + ":" + processDefinition.getVersion()
          + ":" + nextId; // ACT-505
                  
        // ACT-115: maximum id length is 64 charcaters
        if (processDefinitionId.length() > 64) {         
          processDefinitionId = nextId;
        }
</blockcode>

Why y don't use only unique verions or simple UUID, if y need new value every each deploy? This is VERY strange http://kataribe.naist.jp/Activiti/Activiti/commit/9f1772bcaf169f0f25ece0cfef028c0ee3dae4d4 from jbarrezp. Sometime it lead to errors. This BUG (i think) still exist in Activiti 6.0 (!)

[jbarrez]

You seem to have a different kind of humour than me, but who am I to judge.

That line of code was written 6 years ago … I cannot remember at all what was the argumentation back then.

" Sometime it lead to errors. "

Can you eleborate? A unit test demonstrating it going wrong?