Michael Stack

@jojochuang good by you?

> @saintstack why is this PR opened against branch-2.3, not master branch? Habit. Let me put up next PR against master. Thanks for reviews.

Update addresses review comments. Also is more conservative than previous patches (including original posters') regards when to cache filestatus. Posted this patch against master branch too over at https://github.com/apache/hbase/pull/3655

@ramkrish86 Any update here sir?

That is awkward @binlijin With this added level of priority, server B will be able to be processed ahead of server A? And if during processing of B -- assigning...

Ok. You want more feedback? What you think of what has been put here already sir?

Any resolution here. We going to add a flag to bypass new logic?

Kicking @anoopsjohn again!!!

Patch LGTM... Looks like a build bug fix that allows us use mvn -T,--threads so builds can run faster locally for devs... @ayushtkn would you mind saying a bit more...

@ayushtkn makes sense. I was thinking they are probably needed at assembly time because mvn's dependency accounting when threading is likely a little lacking and it looks like these additions...