#idempiere IRC log for Tuesday, 2018-03-06

Not-a748[iDempiereDaily] jenkins built #580 completed (success) http://ci.idempiere.org/job/iDempiereDaily/580/03:34
*** a42niem has joined #idempiere05:22
Not-a748[iDempiere5.1Daily] jenkins built #42 completed (success) http://ci.idempiere.org/job/iDempiere5.1Daily/42/05:32
*** norbertbede has joined #idempiere07:24
*** nmicoud has joined #idempiere07:42
*** CarlosRuiz has joined #idempiere08:59
nmicoudHello CarlosRuiz, thanks for taking time to review my tickets :)09:13
CarlosRuizHi nmicoud - thanks to you for contributing  :-)09:13
nmicoudnow i'm on 5.1, that would be easier to submit patches09:14
nmicoudi'll give you some work :p09:14
norbertbedehi all09:25
norbertbedenmicoud. a question09:25
norbertbedei'm start to use import leads ad_user approx 100k trough csv import09:25
norbertbedei test first 1k records. and as result: error message however approx 100 records was not imported09:26
norbertbedei found the reason. is google map callout on c_location09:26
norbertbedeso as i found in topics, csv import is like window including callouts09:27
norbertbedei cant guarantee quality of leads so adresses sometimes are corrupted. so the question is:09:27
norbertbedeis it better somehow disable for client google coordinate calculation OR change slightly code/review why import skip such issue records09:28
norbertbedewdyt ? maybe this is related to gmail and import csv itself09:28
nmicoudhi norbert09:29
nmicoudi'm not familiar with csv import (i'm still using the 'old' way, ie I_ tables)09:29
nmicoudif you import 100k locations, i fear Google will blacklist you - there should be a limit of requests/day09:30
nmicoudif you want to do the import using csv import, i would advise to do it from Eclipse (there, you should easily deactivate the gps plugin)09:31
norbertbedecan we add sysconfig ?09:32
nmicoudOr maybe add a SysConfig key 'ENABLE_GPS_PLUGIN' that will set to N when importing09:32
nmicoudand then set to Y right after09:32
nmicoudsuer09:32
nmicoud*sure*09:32
norbertbedeyes not nice but helpful for consultant09:32
nmicoudOr perhaps there is a way to kwow if there is a csv import running (in context)09:33
norbertbedemaybe csv import should make hardcoded part avoid to call external ws when improt running09:33
nmicoudyes, something like that09:33
nmicouda trick can be : if Env.getAD_UserID(ctx) > 99999909:33
nmicoudRecords created by SuperUser won't be impacted09:34
nmicoudOr, if you plan to regularly import leads, you can also create a specific user for that purpose and use its userID09:34
norbertbedei prefer make specific behaviour when import - do not call external api's09:35
norbertbedethis is more general and can avoid all use cases in all tables in similar cases09:35
norbertbedewhile external providers will ban us anyway09:36
norbertbedethanks for ideas. make decision.09:38
nmicoudlooking in code to see if we could know data is inserting from csv09:39
nmicoudbut, i see nothing :(09:39
norbertbedeCarlosRuiz whyt to add generic DISABLE_API_CALLS_CSV_IMPORT sysconfig09:40
CarlosRuizHi norbertbede - nmicoud09:47
CarlosRuizDISABLE_API_CALLS_CSV_IMPORT - it doesn't sound good as a generic thing09:47
CarlosRuizit sounds is specific for every provider09:47
CarlosRuizI think the suggestion from nmicoud about ENABLE_GPS_PLUGIN sounds better09:47
norbertbedeok. however admin  - mainly - in cloud environment must be informed well, else suck when start import a huge data set09:48
norbertbedewhen installation admin <> tenant admin let say "Tenant Power user"09:49
norbertbedethanks for comments !!!09:56
*** norbertbede has quit IRC10:44
*** dagelf_ has quit IRC15:01
*** CarlosRuiz has quit IRC15:04
*** dagelf has joined #idempiere15:15
*** CarlosRuiz has joined #idempiere15:17
*** nmicoud has quit IRC16:31
*** CarlosRuiz has quit IRC19:17
*** CarlosRuiz has joined #idempiere19:33
*** CarlosRuiz has quit IRC20:25
*** CarlosRuiz has joined #idempiere20:37
*** CarlosRuiz has quit IRC20:45
*** a42niem has quit IRC23:10

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!