Skip to content

[Bug][RoutineLoad] Fix bug that routine Load encounter "label already used" exception#2959

Merged
morningman merged 3 commits intoapache:masterfrom
morningman:routine_load_submit
Feb 22, 2020
Merged

[Bug][RoutineLoad] Fix bug that routine Load encounter "label already used" exception#2959
morningman merged 3 commits intoapache:masterfrom
morningman:routine_load_submit

Conversation

@morningman
Copy link
Contributor

@morningman morningman commented Feb 21, 2020

This CL modify 2 things:

  1. When a routine load task submit failed, it will not be put back to the task queue.
  2. The rpc timeout when executing a routine load task in BE is set to query_timeout of the task plan.

ISSUE: #2964

partitions = routineLoadJob.getPartitions() == null ? null : Joiner.on(",").join(routineLoadJob.getPartitions());
strictMode = routineLoadJob.isStrictMode();
timezone = routineLoadJob.getTimezone();
timeout = (int) routineLoadJob.getMaxBatchIntervalS() * 2;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does FE send the timeout property to BE in query_options?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, previous we just use the Config.stream_load_default_timeout_second for the routine load task.

@morningman morningman changed the title Routine load submit [Bug][RoutineLoad] Fix bug that routine Load encounter "label already used" exception Feb 21, 2020
@morningman morningman merged commit 8eb413f into apache:master Feb 22, 2020
morningman added a commit to baidu-doris/incubator-doris that referenced this pull request Mar 11, 2020
… used" exception (apache#2959)

This CL modify 2 things:

1. When a routine load task submit failed, it will not be put back to the task queue.
2. The rpc timeout when executing a routine load task in BE is set to `query_timeout` of the task plan.

ISSUE: apache#2964
songenjie pushed a commit to songenjie/incubator-doris that referenced this pull request Apr 8, 2020
…counter "label already used" exception (apache#2959)

This CL modify 2 things:

1. When a routine load task submit failed, it will not be put back to the task queue.
2. The rpc timeout when executing a routine load task in BE is set to `query_timeout` of the task plan.

ISSUE: apache#2964
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants