Opened 4 years ago

Closed 3 years ago

#365 closed task (wontfix)

expand parameter should expand everything by default

Reported by: dmisev Owned by: pcampalani
Priority: minor Milestone: 9.0.x
Component: secore Version: 8.4
Keywords: expand default Cc: pcampalani
Complexity: Easy

Description (last modified by pcampalani)

The default for expand in SECORE should be

expand=full

Change History (8)

comment:1 Changed 4 years ago by dmisev

  • Milestone changed from 8.5 to 9.0-alpha1

comment:2 Changed 4 years ago by abeccati

  • Milestone changed from 9.0-beta1 to 9.0

Milestone 9.0-beta1 deleted

comment:3 Changed 3 years ago by pcampalani

  • Description modified (diff)

comment:4 Changed 3 years ago by pcampalani

  • Owner changed from dmisev to pcampalani
  • Status changed from new to assigned

comment:5 Changed 3 years ago by pcampalani

  • Complexity changed from Medium to Easy
  • Keywords expand default added
  • Status changed from assigned to accepted

comment:6 Changed 3 years ago by pcampalani

Patch submitted, however while fixing #806 I realize that a default full expansion is heavily killing the performance of SECORE: I'll revert back to a default value of 2.

comment:7 Changed 3 years ago by pcampalani

For better tuning, I also suggest as new feature to have two expansion parameters for configuring both the XML tree level and the recursion level.

comment:8 Changed 3 years ago by pcampalani

  • Resolution set to wontfix
  • Status changed from accepted to closed

Closing this, as full recursive link expansion kills SECORE performance (in my local machine sometimes even inhibiting to get a response).
In case this will be reconsidered, the patch for setting full expansion is changeset:4aeba67 (it has been reverted in changeset:76e97d8).

Note: See TracTickets for help on using tickets.