Gaurav Aggarwal logiclord

  • Joined on
@logiclord
@logiclord master is now 1c4796c
@logiclord
@logiclord
Catch exceptions in individual triggers and avoid complete break down of Azkaban schedule
logiclord commented on pull request azkaban/azkaban#445
@logiclord

Thanks John. I have added the comment.

@logiclord
  • @logiclord c6a5346
    Adding comment explicitly to highlight missed trigger
@logiclord
Enhance flow runner to prioritize ready jobs using some property
logiclord commented on pull request azkaban/azkaban#445
@logiclord

Issue #446

@logiclord
Catch exceptions in individual triggers and avoid complete break down of Azkaban schedule
logiclord opened pull request azkaban/azkaban#445
@logiclord
Catch exceptions in individual triggers
1 commit with 29 additions and 32 deletions
@logiclord
logiclord commented on pull request azkaban/azkaban#443
@logiclord

Minor comment. Otherwise LGTM

logiclord commented on commit hluu/azkaban2@41a1bb9226
@logiclord

We don't need this if statement logger level should take care of stuff ?

logiclord commented on pull request azkaban/azkaban#443
@logiclord

This is a big change. We should add this in http://azkaban.github.io/azkaban/docs/

logiclord commented on pull request azkaban/azkaban#443
@logiclord

Just had a look tests seems to be good examples,

logiclord commented on pull request azkaban/azkaban#443
@logiclord

Just had a look tests seems to be good examples,

logiclord commented on pull request azkaban/azkaban#443
@logiclord

logger.debug ?

logiclord commented on pull request azkaban/azkaban#443
@logiclord

Should we have an upper limit on body size ?

logiclord commented on pull request azkaban/azkaban#443
@logiclord

Do you need to resolve props ? Azkaban should have done it before starting execution ?

logiclord commented on pull request azkaban/azkaban#443
@logiclord

Will be good to have some examples ?

logiclord commented on pull request azkaban/azkaban#443
@logiclord

replaceFirst ?

logiclord commented on pull request azkaban/azkaban#441
@logiclord

LGTM

@logiclord
logiclord commented on pull request azkaban/azkaban#431
@logiclord

LGTM

logiclord commented on pull request azkaban/azkaban#431
@logiclord

One suggestion. It will be good to test on a cluster, interface changes might break some dependency of azkaban-plugins on azkaban.

logiclord commented on pull request azkaban/azkaban#431
@logiclord

How about using project Id ? Project names can be re-assigned if original owner deletes existing project with a given name and another user create …

@logiclord