falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pallavi Rao" <pallavi....@inmobi.com>
Subject Re: Review Request 41203: FALCON-1572 Only one instance is running in a process when run using Native Scheduler
Date Mon, 14 Dec 2015 10:14:40 GMT


> On Dec. 14, 2015, 8:44 a.m., pavan kumar kolamuri wrote:
> > scheduler/src/main/java/org/apache/falcon/execution/NotificationHandler.java, line
32
> > <https://reviews.apache.org/r/41203/diff/2/?file=1159977#file1159977line32>
> >
> >     Just curious 0,3,5 are given so that to add more priorities right ? Then i feel
0,5,10 might be better to accomodate more priorities
> 
> Pallavi Rao wrote:
>     <p>No particular reason :-). It could be 3,2,1 or 300, 250, 100. As long as
high has highest number and the number goes down with priority it is fine. If and when we
want more priorities, we can always change the numbers.</p>
> 
> pavan kumar kolamuri wrote:
>     During migration we have seen issues because of changing numbers in enum. As long
as it is taken care no issues.

Okie. I will change it 10, 5, 0 to give enough lee-way. Though don't expect to see so many
levels of priorities.


- Pallavi


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41203/#review110179
-----------------------------------------------------------


On Dec. 11, 2015, 9:24 a.m., Pallavi Rao wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41203/
> -----------------------------------------------------------
> 
> (Updated Dec. 11, 2015, 9:24 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1572
>     https://issues.apache.org/jira/browse/FALCON-1572
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Wrong Datastructure was used to load and store into cache.
> 
> 
> Diffs
> -----
> 
>   scheduler/pom.xml 336997d 
>   scheduler/src/main/java/org/apache/falcon/execution/EntityExecutor.java 88d88c1 
>   scheduler/src/main/java/org/apache/falcon/execution/ExecutionInstance.java 5f96d3f

>   scheduler/src/main/java/org/apache/falcon/execution/FalconExecutionService.java b6741a4

>   scheduler/src/main/java/org/apache/falcon/execution/NotificationHandler.java b071f5f

>   scheduler/src/main/java/org/apache/falcon/notification/service/impl/JobCompletionService.java
23f2b4e 
>   scheduler/src/main/java/org/apache/falcon/notification/service/impl/SchedulerService.java
ace8444 
>   scheduler/src/test/java/org/apache/falcon/notification/service/SchedulerServiceTest.java
c43ccf0 
> 
> Diff: https://reviews.apache.org/r/41203/diff/
> 
> 
> Testing
> -------
> 
> Manual testing done. Once ITs are checked in, will add a test case for this too.
> 
> 
> Thanks,
> 
> Pallavi Rao
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message