int-basic-pgsql - Build # 1798 - Fixed:

Check console output at https://ci.openbravo.com/job/int-basic-pgsql/1798/ to 
view the results.


Committers since last success:

Changes for Build #1797

    Víctor Martínez Romanos <[email protected]> null
    Related to issue 35205: new normal index 
C_CONVERSION_RATE_DOCUMENT.C_INVOICE_ID
Added this new normal indexes because it has been detected to avoid seq. scan 
in several flows.
More info at 
https://docs.google.com/spreadsheets/d/1m5GZaaoEkYvyGdyxddtxxb3SeAhvtavfdq53iumOT7s
        src-db/database/model/tables/C_CONVERSION_RATE_DOCUMENT.xml

    Víctor Martínez Romanos <[email protected]> null
    Fixed issue 35205: Partial indexes functional stuff

The only performance drawback in transforming a normal index to a partial index 
is in the queries with a where clause filtering by a indexed column when this 
column is null.
In this case, and only when the ratio of null values is very low, the query 
will probably execute a seq. scan over the table instead of using the index, 
thus killing the performance.

To detect this scenario, we have "greped" the PI code for "is*null" string 
(case insensitve), and filtered the places where the candidate indexes appear. 
Then, we have individually analize each of the queries to detect whether we 
could safely transform the index to partial index based on the previous 
explanation.

The result can be found in 
https://docs.google.com/spreadsheets/d/1m5GZaaoEkYvyGdyxddtxxb3SeAhvtavfdq53iumOT7s
 Functional Analysis tab.
Values with Candidate = Y are the ones to be transformed to partial indexes, 
Candidate = N means to keep the normal index, and TBCN is to create a normal 
index because it would help in performance.
When there is no query that uses this index inside an "IS NULL" condition, we 
have automatically set it to partial index as there shouldn't be any risk
For the rest of the candidates there are individual explanations which can be 
found as a note (in the Candidate column and/or in the places where this index 
might be used).

For future references, this is the command used to grep the PI code:
grep -n --color=always  -riI  "is[[:space:]]*null" --exclude \*.js 
--exclude-dir src-wad --exclude-dir build  /home/openbravo/HG/erp/devel/pi | 
grep -vi case | more
        
modules/org.openbravo.userinterface.selector/src-db/database/model/tables/OBUISEL_SELECTOR_TRL.xml
        src-db/database/model/tables/AD_USER.xml
        src-db/database/model/tables/A_AMORTIZATIONLINE.xml
        src-db/database/model/tables/A_ASSET.xml
        src-db/database/model/tables/C_ACCTSCHEMA_PROCESS.xml
        src-db/database/model/tables/C_BPARTNER.xml
        src-db/database/model/tables/C_CITY.xml
        src-db/database/model/tables/C_CONVERSION_RATE_DOCUMENT.xml
        src-db/database/model/tables/C_INVOICE.xml
        src-db/database/model/tables/C_INVOICELINE.xml
        src-db/database/model/tables/C_INVOICETAX_CASHVAT.xml
        src-db/database/model/tables/C_ORDER.xml
        src-db/database/model/tables/C_ORDERLINE.xml
        src-db/database/model/tables/C_POC_DOCTYPE_TEMPLATE.xml
        src-db/database/model/tables/C_POC_EMAILDEFINITION.xml
        src-db/database/model/tables/C_TAXREGISTER.xml
        src-db/database/model/tables/C_TAXREGISTERLINE.xml
        src-db/database/model/tables/C_TAXREGISTER_TYPE_LINES.xml
        src-db/database/model/tables/C_TAX_ZONE.xml
        src-db/database/model/tables/FIN_FINACC_TRANSACTION.xml
        src-db/database/model/tables/FIN_ORIG_PAYMENT_SCHEDULE.xml
        src-db/database/model/tables/FIN_ORIG_PAYM_SCHEDDETAIL.xml
        src-db/database/model/tables/FIN_PAYMENT_SCHEDULE.xml
        src-db/database/model/tables/FIN_PAYMENT_SCHEDULEDETAIL.xml
        src-db/database/model/tables/GL_JOURNAL.xml
        src-db/database/model/tables/MA_MAINTENANCE.xml
        src-db/database/model/tables/MA_MEASURE_GROUP.xml
        src-db/database/model/tables/MA_PROCESS.xml
        src-db/database/model/tables/MA_WEINCIDENCE.xml
        src-db/database/model/tables/MRP_RUN_PRODUCTIONLINE.xml
        src-db/database/model/tables/M_COSTADJUSTMENTLINE.xml
        src-db/database/model/tables/M_COSTING.xml
        src-db/database/model/tables/M_INOUT.xml
        src-db/database/model/tables/M_INOUTLINE.xml
        src-db/database/model/tables/M_LANDEDCOST.xml
        src-db/database/model/tables/M_LC_COST.xml
        src-db/database/model/tables/M_LC_MATCHED.xml
        src-db/database/model/tables/M_LC_RECEIPT.xml
        src-db/database/model/tables/M_LC_RECEIPTLINE_AMT.xml
        src-db/database/model/tables/M_LOCATOR.xml
        src-db/database/model/tables/M_MATCHPO.xml
        src-db/database/model/tables/M_PRODUCT.xml
        src-db/database/model/tables/M_RAPPEL_BPARTNER.xml
        src-db/database/model/tables/M_RAPPEL_INVOICE.xml
        src-db/database/model/tables/M_RAPPEL_PRODUCT.xml
        src-db/database/model/tables/M_RAPPEL_PRODUCTCATEGORY.xml
        src-db/database/model/tables/M_RAPPEL_SCALE.xml
        src-db/database/model/tables/M_RESERVATION.xml
        src-db/database/model/tables/M_RESERVATION_STOCK.xml
        src-db/database/model/tables/M_STOCK_AUX.xml
        src-db/database/model/tables/M_STOCK_VALUATION.xml
        src-db/database/model/tables/M_TRANSACTION.xml
        src-db/database/model/tables/M_TRANSACTION_COST.xml
        src-db/database/model/tables/M_WAREHOUSE_SHIPPER.xml
        src-db/database/model/tables/S_TIMEEXPENSELINE.xml

Changes for Build #1798

    Víctor Martínez Romanos <[email protected]> null
    Related to issue 35205: reverted some partial indexes

Some indexes have been reverted to normal indexes because they belong to high 
volume tables and the benefit to transform them to partial index is very 
reduced (if any) because most of the values will usually be not null.
Besides, transforming them to partial index would create a big delay in the 
update database process.
        src-db/database/model/tables/A_AMORTIZATIONLINE.xml
        src-db/database/model/tables/C_INVOICELINE.xml
        src-db/database/model/tables/C_INVOICETAX_CASHVAT.xml
        src-db/database/model/tables/C_ORDERLINE.xml
        src-db/database/model/tables/C_TAX_ZONE.xml
        src-db/database/model/tables/FIN_FINACC_TRANSACTION.xml
        src-db/database/model/tables/M_INOUTLINE.xml
        src-db/database/model/tables/M_LANDEDCOST.xml

    Martin Taal <[email protected]> null
    Related to issue 35326: Support basic authentication
back out of changeset as it causes several retail offline tests to fail
        src/org/openbravo/base/secureApp/HttpSecureAppServlet.java




Last 20 lines of the console output:

[...truncated 19647 lines...]
Total time: 3 seconds

executing script 'Tomcat stop'
[int-basic-pgsql] $ /bin/bash /tmp/build_step_template7393737016625125240.sh 
ENABLE
DEBUG: Tomcat stop called with parameters: ENABLED=ENABLE
 * Stopping Tomcat servlet engine for Openbravo tomcat
   ...done.

executing script 'Check openbravo log'
[int-basic-pgsql] $ /bin/bash /tmp/build_step_template2984036812788952434.sh 
ENABLE
DEBUG: Check openbravo log called with parameters: ENABLED=ENABLE
Check log: openbravo log (artifact: int-basic-pgsql.log) is clean

Recording test results
Archiving artifacts
Checking \] ERROR|\] WARN
Email was triggered for: Fixed
Trigger Success was overridden by another trigger and will not send an email.
Sending email for trigger: Fixed
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Openbravo-builds mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-builds

Reply via email to