Skip to content
Snippets Groups Projects
  1. Jun 20, 2013
  2. Jun 19, 2013
  3. Jun 18, 2013
    • Konstantin Osipov's avatar
      Merge branch 'tuple-update' · 8a6400f5
      Konstantin Osipov authored
      8a6400f5
    • Konstantin Osipov's avatar
      tuple-access: code review fixes · af4164fb
      Konstantin Osipov authored
      Two significant changes (both arguable, since this patch is about
      taste by a large measure):
      
      1) Split tuple_seek() into two: tuple_rewind()
      and tuple_seek().
      
      tuple_rewind() simply initializes the iterator and
      sets it to 'before-first' position.
      tuple_seek() returns a field, and sets the iterator
      to the position after the returned field.
      
      Previosly there was only tuple_seek(), which required
      tuple_next() to get the field, and it was confusing whether
      or not tuple_next() after tuple_seek() is required.
      Indeed, the very code I chagned sometimes would
      call tuple_next() after tuple_seek() and sometimes would not.
      
      2) Move the return value of the iterator outside
      of iteration state.
      
      The advantage of "binding" the out parameters of the iterator to
      iterator state is that you bind once, and don't need to supply out
      parameters into every call to tuple_next() function. It also makes
      it easy to add extra out parameters for bind in the future (field
      data type, flags, etc).
      
      This makes "binding" technique popular in database driver
      APIs. Binding makes API more stable, which is also
      important in drivers.
      
      In this case, however, binding adds for extra lines of code
      and (possibly) few extra instructions. Plus, since
      this is an internal API which we can change any day,
      its stability is not as important. So it seems that for
      now we're better of with an API which is a bit
      more concise/efficient, and can switch to using binding later,
      if needed.
      af4164fb
    • Roman Tsisyk's avatar
      Split off box.update code from request.cc · ea791469
      Roman Tsisyk authored
      ea791469
    • Dmitry E. Oboukhov's avatar
      a3767dcc
  4. Jun 14, 2013
  5. Jun 13, 2013
  6. Jun 11, 2013
  7. Jun 10, 2013
  8. Jun 08, 2013
  9. Jun 07, 2013
Loading