You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by da...@apache.org on 2026/01/15 17:06:16 UTC

(doris) branch routineload_flexible_update updated (d25ff385787 -> ee70fcde3d4)

This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a change to branch routineload_flexible_update
in repository https://gitbox.apache.org/repos/asf/doris.git


    from d25ff385787 fix: Resolve code review issues for flexible partial update
     add b068217c443 fix: Move static methods after constructor to fix DeclarationOrder checkstyle
     add f7d734d458d fix: Fix compilation errors - wrap UserException and use columnDescs field
     add 93bcfcd7f49 test: Add unit tests for flexible partial update in routine load
     add 7931212a840 fix: Fix unit tests to avoid NPE from gsonPostProcess
     add b90147e0110 fix: Update error messages to match expected format in existing tests
     new ee70fcde3d4 fix: Fix test assertions and Groovy escaping in routine load tests

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../java/org/apache/doris/catalog/OlapTable.java   |  12 +-
 .../doris/load/routineload/RoutineLoadJob.java     |   5 +-
 .../plans/commands/info/CreateRoutineLoadInfo.java |  76 +++++++------
 .../doris/load/routineload/RoutineLoadJobTest.java | 125 +++++++++++++++++++++
 ...est_routine_load_flexible_partial_update.groovy |   6 +-
 ...ine_load_partial_update_new_key_behavior.groovy |   2 +-
 6 files changed, 177 insertions(+), 49 deletions(-)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


(doris) 01/01: fix: Fix test assertions and Groovy escaping in routine load tests

Posted by da...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch routineload_flexible_update
in repository https://gitbox.apache.org/repos/asf/doris.git

commit ee70fcde3d445f10c2f1ae6a0d162d28348cf78a
Author: Yongqiang YANG <da...@gmail.com>
AuthorDate: Thu Jan 15 09:06:02 2026 -0800

    fix: Fix test assertions and Groovy escaping in routine load tests
    
    Update error message assertion to match actual code behavior and fix
    Groovy string escaping for jsonpaths to prevent token recognition errors.
---
 .../routine_load/test_routine_load_flexible_partial_update.groovy       | 2 +-
 .../test_routine_load_partial_update_new_key_behavior.groovy            | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/regression-test/suites/load_p0/routine_load/test_routine_load_flexible_partial_update.groovy b/regression-test/suites/load_p0/routine_load/test_routine_load_flexible_partial_update.groovy
index 2aa3e80f3a0..f7f99a1d5fd 100644
--- a/regression-test/suites/load_p0/routine_load/test_routine_load_flexible_partial_update.groovy
+++ b/regression-test/suites/load_p0/routine_load/test_routine_load_flexible_partial_update.groovy
@@ -273,7 +273,7 @@ suite("test_routine_load_flexible_partial_update", "nonConcurrent") {
                 (
                     "max_batch_interval" = "10",
                     "format" = "json",
-                    "jsonpaths" = '["\\$.id", "\\$.name", "\\$.score"]',
+                    "jsonpaths" = '[\"\$.id\", \"\$.name\", \"\$.score\"]',
                     "unique_key_update_mode" = "UPDATE_FLEXIBLE_COLUMNS"
                 )
                 FROM KAFKA
diff --git a/regression-test/suites/load_p0/routine_load/test_routine_load_partial_update_new_key_behavior.groovy b/regression-test/suites/load_p0/routine_load/test_routine_load_partial_update_new_key_behavior.groovy
index a6a97253e98..eab01074ff3 100644
--- a/regression-test/suites/load_p0/routine_load/test_routine_load_partial_update_new_key_behavior.groovy
+++ b/regression-test/suites/load_p0/routine_load/test_routine_load_partial_update_new_key_behavior.groovy
@@ -276,7 +276,7 @@ suite("test_routine_load_partial_update_new_key_behavior", "nonConcurrent") {
                     "property.kafka_default_offsets" = "OFFSET_BEGINNING"
                 );
             """
-            exception "partial_update_new_key_behavior can only be set when partial_columns is true"
+            exception "partial_update_new_key_behavior can only be set when partial update is enabled"
         }
     }
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org