Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/23789

to look at the new patch set (#2).

Change subject: IMPALA-14535: Improve wait for HMS events sync with 
hierarchical event processing
......................................................................

IMPALA-14535: Improve wait for HMS events sync with hierarchical event 
processing

With hierarchical event processing, MetastoreEventsProcessor's
lastSyncedEventId_ represents the last event dispatched to the
EventExecutorService.Events are queued to appropriate
DbEventExecutors and/or TableEventExecutor for processing.
And events are removed from these queues only after they
are processed by the respective executor threads.

When a wait for HMS events sync request is issued, the
following sequence is executed:
1. The current notification event id is fetched from HMS.
2. MetastoreEventsProcessor is ensured to fetch and dispatch
   all events up to and including this notification event id
   to the EventExecutorService.
3. The list of databases and tables requiring synchronization
   is extracted from the request.
4. To determine whether synchronization for a required
   database and or table is complete, relevant DbEventExecutor
   and TableEventExecutor instances are checked directly.
   Synchronization is considered complete only when the
   event-processing progress of the required DbProcessor and
   TableProcessor instances has surpassed the captured
   notification event id.

This approach guarantees that all HMS events that existed at
the time of sync request have been fully processed for the
requested databases and tables.

Testing:
    - Added an FE test and ran the existing tests.

Change-Id: I55cea4cb8e04860202e56e1b1bf2596613b4946c
---
M be/src/catalog/catalog-server.cc
M fe/src/main/java/org/apache/impala/catalog/events/DbEventExecutor.java
M fe/src/main/java/org/apache/impala/catalog/events/EventExecutorService.java
M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
M 
fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java
M fe/src/main/java/org/apache/impala/catalog/events/TableEventExecutor.java
M 
fe/src/test/java/org/apache/impala/catalog/events/EventExecutorServiceTest.java
7 files changed, 362 insertions(+), 40 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/89/23789/2
--
To view, visit http://gerrit.cloudera.org:8080/23789
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I55cea4cb8e04860202e56e1b1bf2596613b4946c
Gerrit-Change-Number: 23789
Gerrit-PatchSet: 2
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>

Reply via email to