D1358: remotenames: store journal entry for bookmarks if journal is loaded

[quark (Jun Wu)]

quark added a comment.


  In https://phab.mercurial-scm.org/D1358#23388, @smf wrote:
  
  > 1. Create a v1 storage format api to hold this information (maybe start with sqlite or something very simple at first for the backend)
  
  
  On sqlite:
  
  - Some people use traditional tools like `grep`, `less` on `blackbox.log`. Using sqlite would make blackbox harder to use.
  
  - sqlite calls `fsync` by default. It's a performance issue under heavy I/O.
  
  > 2. Create a helper utility (decorator, or just wrap ui, etc)
  > 3. Use helper utility on all these commands to keep the data extension-agnostic
  
  The current APIs (ui.log and repo.journal.record) do not look
  terrible. Could you give an example of how this diff could be
  improved with your proposed new API? i.e. is it becoming shorter,
  less cryptic?
  
  If the code under the new API look similar to the existing one,
  I wonder whether they could be implemented independently - i.e.
  this series continue using the existing API and changing the
  format is independent from this series.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D1358

To: pulkit, #hg-reviewers
Cc: quark, durin42, smf, dlax, mercurial-devel