Project

General

Profile

Actions

User story #3451

closed

User story #3450: Improve perfs for technique Download file from shared folder

Read data from external file in download from shared folder

Added by Nicolas CHARLES over 11 years ago. Updated over 9 years ago.

Status:
Rejected
Priority:
1 (highest)
Category:
Techniques
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

we need to read data from an external file, rather that writing a big array, in the Technique download from shared folder
It'll iprove readability, and performance

Actions #1

Updated by Nicolas CHARLES over 11 years ago

  • Status changed from In progress to 10
  • Assignee changed from Nicolas CHARLES to Jonathan CLARKE
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/85
Actions #2

Updated by Matthieu CERDA over 11 years ago

  • Target version changed from 2.3.12 to 2.3.13
Actions #3

Updated by Nicolas PERRON over 11 years ago

  • Target version changed from 2.3.13 to 84
Actions #4

Updated by Nicolas PERRON over 11 years ago

  • Target version changed from 84 to 2.4.7
Actions #5

Updated by Nicolas PERRON over 11 years ago

  • Target version changed from 2.4.7 to 2.4.8
Actions #6

Updated by Jonathan CLARKE about 11 years ago

  • Status changed from 10 to Discussion
  • Assignee changed from Jonathan CLARKE to Nicolas CHARLES

I seem to recall that this has an undesirable side effect, making it impossible to use variables in any of the parameters here.

What shall we conclude? Should we reject this?

Actions #7

Updated by Nicolas CHARLES about 11 years ago

  • Status changed from Discussion to Rejected

Yeah, that's the problem
We should indeed probably reject it, as long as we don't have better solutions than that

Actions #8

Updated by Benoît PECCATTE over 9 years ago

  • Category set to Techniques
Actions #9

Updated by Benoît PECCATTE over 9 years ago

  • Tracker changed from Enhancement to User story
Actions

Also available in: Atom PDF