dengzhhu653 commented on code in PR #5542:
URL: https://github.com/apache/hive/pull/5542#discussion_r2009464909


##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetastoreDirectSqlUtils.java:
##########
@@ -152,9 +154,18 @@ static <T> int loopJoinOrderedResult(PersistenceManager 
pm, TreeMap<Long, T> tre
           if (nestedId < id) {
             throw new MetaException("Found entries for unknown ID " + 
nestedId);
           }
-          if (nestedId > id)
+          if (nestedId > id) {
+            if (!tree.containsKey(nestedId)) {
+              Throwable throwable = (new Throwable()).fillInStackTrace();

Review Comment:
   it helps to know what the `many` is (one-to-many mapping), for example if 
the `value` is Partition, then the `many` can be parameters or partition values 
by the stack call from `throwable`.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to