SAP/abapGit
1
0
Fork 0
mirror of https://github.com/abapGit/abapGit.git synced 2025-05-01 12:20:51 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Don't run code inspector in parallel when disabled (#3348)

Browse Source 
* Don't run code inspector in parallel when disabled

* Restructure if

* Fix spacing

* Fix comment position
This commit is contained in:
Frederik Hudák 2020-05-07 11:30:35 +02:00 committed by GitHub
parent 14a2820d30
commit a64b6d96f8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 32 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
src/zcl_abapgit_code_inspector.clas.abap
View File

@ -40,14 +40,16 @@ CLASS zcl_abapgit_code_inspector DEFINITION
PRIVATE SECTION. PRIVATE SECTION.
DATA mv_success TYPE abap_bool . DATA mv_success TYPE abap_bool .
TYPES: t_run_mode TYPE sychar01.
CONSTANTS: CONSTANTS:
BEGIN OF co_run_mode, BEGIN OF co_run_mode,
run_with_popup TYPE sychar01 VALUE 'P', run_with_popup TYPE t_run_mode VALUE 'P',
run_after_popup TYPE sychar01 VALUE 'A', run_after_popup TYPE t_run_mode VALUE 'A',
run_via_rfc TYPE sychar01 VALUE 'R', run_via_rfc TYPE t_run_mode VALUE 'R',
run_in_batch TYPE sychar01 VALUE 'B', run_in_batch TYPE t_run_mode VALUE 'B',
run_loc_parallel TYPE sychar01 VALUE 'L', run_loc_parallel TYPE t_run_mode VALUE 'L',
run_direct TYPE sychar01 VALUE 'L', run_direct TYPE t_run_mode VALUE 'L',
END OF co_run_mode . END OF co_run_mode .
DATA mo_inspection TYPE REF TO cl_ci_inspection . DATA mo_inspection TYPE REF TO cl_ci_inspection .
DATA mv_name TYPE sci_objs . DATA mv_name TYPE sci_objs .
@ -66,12 +68,16 @@ CLASS zcl_abapgit_code_inspector DEFINITION
zcx_abapgit_exception . zcx_abapgit_exception .
METHODS create_inspection METHODS create_inspection
IMPORTING IMPORTING
!io_set TYPE REF TO cl_ci_objectset io_set TYPE REF TO cl_ci_objectset
!io_variant TYPE REF TO cl_ci_checkvariant io_variant TYPE REF TO cl_ci_checkvariant
RETURNING RETURNING
VALUE(ro_inspection) TYPE REF TO cl_ci_inspection VALUE(ro_inspection) TYPE REF TO cl_ci_inspection
RAISING RAISING
zcx_abapgit_exception . zcx_abapgit_exception .
METHODS decide_run_mode
RETURNING
VALUE(rv_run_mode) TYPE t_run_mode.
ENDCLASS. ENDCLASS.
@ -125,13 +131,7 @@ CLASS zcl_abapgit_code_inspector IMPLEMENTATION.
" Because we want to persist them so we can run it in parallel. " Because we want to persist them so we can run it in parallel.
" Both are deleted afterwards. " Both are deleted afterwards.
mv_name = |{ sy-uname }_{ sy-datum }_{ sy-uzeit }|. mv_name = |{ sy-uname }_{ sy-datum }_{ sy-uzeit }|.
mv_run_mode = decide_run_mode( ).
" We have to disable parallelization in batch because of lock errors.
IF sy-batch = abap_true.
mv_run_mode = co_run_mode-run_via_rfc.
ELSE.
mv_run_mode = co_run_mode-run_loc_parallel.
ENDIF.
ENDMETHOD. ENDMETHOD.
@ -339,4 +339,21 @@ CLASS zcl_abapgit_code_inspector IMPLEMENTATION.
ENDTRY. ENDTRY.
ENDMETHOD. ENDMETHOD.
METHOD decide_run_mode.
DATA: lo_settings TYPE REF TO zcl_abapgit_settings.
lo_settings = zcl_abapgit_persist_settings=>get_instance( )->read( ).
IF sy-batch = abap_true.
" We have to disable parallelization in batch because of lock errors.
rv_run_mode = co_run_mode-run_via_rfc.
ELSEIF lo_settings->get_parallel_proc_disabled( ) = abap_false.
rv_run_mode = co_run_mode-run_loc_parallel.
ELSE.
rv_run_mode = co_run_mode-run_via_rfc.
ENDIF.
ENDMETHOD.
ENDCLASS. ENDCLASS.