Distinct is not supported by dbtng

As you can read in the error message. This feature is not support by dbtng at the moment.

.

Really, this is more like "Can't fix", but once dbtng supports it, this might need revisiting.

I have a similar problem.

There's concluded views in which the same information.
If I set the value of Distinct: Yes, it appears a similar error:

(D7-dev, Views-dev)

No field level distinct should have been filed against core. Now that it has this should be postponed until that is fixed. See #975936: No Field Level Distinct Query Support

This is a Views bug.

*I* added that exception to the code. Column-level distinct is a myth. What Views needs to use is a GROUP BY, and Views needs to decide what to do with the non-grouped fields.

As Dave Reid pointed out, Views already adds a GROUP BY in that case. Fixing the immediate bug should then be as easy as this.

query->distinct() is never called.
Uncommented the stuff out and moved a bit around.

This works for me.

Just as a notice, groupby and distinct together removes the "distinct"-feature by adding the base field to the groupby.

-      $base_field_alias = $this->add_field($this->base_table, $this->base_field, NULL, array('distinct' => TRUE));
+      $base_field_alias = $this->add_field($this->base_table, $this->base_field);
+      $query->distinct();
       $this->add_groupby($base_field_alias);

The point is: we don't want nor need this. The add_groupby is all that matters.

     // Make the query distinct if the option was set.
     if (!empty($this->options['distinct'])) {
-      // $this->set_distinct();
+      $this->set_distinct();
     }

Hm. What explains that clicking on this damn checkbox in the UI does something then?

$this->set_distinct set's $this->distinct

And

    if (empty($this->no_distinct) && $this->distinct && !empty($this->fields)) {
      $base_field_alias = $this->add_field($this->base_table, $this->base_field, NULL, array('distinct' => TRUE));
      $this->add_groupby($base_field_alias);
    }

needs this flag.

So what about this patch.

Does this mean no row level distinct support in Views? If not, how will distinct be handled?

This worked great guys, thanks!

Thanks

Commited to the 7.x branch.