message: feedback round
diff: @@ -1334,12 +1334,14 @@ public:
  void FlushDictionary(BasicColumnWriterState &state_p, ColumnWriterStatistics *stats_p) override {
  auto &stats = (StringStatisticsState &)*stats_p;
  auto &state = (StringColumnWriterState &)state_p;
  - if (state.IsDictionaryEncoded()) {
  + if (!state.IsDictionaryEncoded()) {
  + return;
  + }
  // first we need to sort the values in index order
  auto values = vector<string>(state.dictionary.size());
  - for (auto iter = state.dictionary.cbegin(); iter != state.dictionary.cend(); iter++) {
  - D_ASSERT(values[iter->second].empty());
  - values[iter->second] = iter->first;
  + for (const auto &entry : state.dictionary) {
  + D_ASSERT(values[entry.second].empty());
  + values[entry.second] = entry.first;
  }
  // first write the contents of the dictionary page to a temporary buffer
  auto temp_writer = make_unique<BufferedSerializer>();
  @@ -1354,7 +1356,6 @@ public:
  // flush the dictionary page and add it to the to-be-written pages
  WriteDictionary(state, move(temp_writer), values.size());
  }
  - }
  idx_t GetRowSize(Vector &vector, idx_t index, BasicColumnWriterState &state_p) override {
  auto &state = (StringColumnWriterState &)state_p;
  