asterite / rgviz Goto Github PK

The is null constraint is case sensitive. When it is all uppercase, I receive an invalid query. Seems to only work with all lower case. Not sure if this is a bug as I am fine doing all lower case, but it was at least unexpected behavior for me.

When I submit a query such as SELECT * WHERE created_at >= datetime 'March 08, 2012 03:00:00' I would expect the parsing of that timestamp to respect the time zone set in Time.zone.

I tried to fix this first in rgviz-rails Rgviz::Executor#column_value before realizing this was the wrong method :( The method I am looking for is Rgviz::Parser#parse_atomic_column but I am not sure how to fix it. Time.zone seems to be a rails enhancement and rgviz is supposed to work without rails. So, I think the fix should go in rgviz-rails, but I am not coming up with a good way to do it. Thoughts?

In memory_executor.rb line 15 when execute is defined. The definition was changed from:

def execute(query, options = {})

to:

def execute(query)

This causes line 30 of init.rb in rgviz-rails to fail because it is passing the opts parameter which causes an "Error in query: wrong number of arguments (2 for 1)" error. Please include the options parameter back in to be consistent with the other executors. Thanks.

Is it posisble to a make a query that do a sum with if inside?

For example :

SUM(if( users.origin IS NULL, 1,0))

Checked the documentation but don't find the if statement, there is a workaround?

If I have a query SELECT * WHERE column is null, the resulting SQL looks like: SELECT * FROM table WHERE "table"."column"is null Notice that there is no space between the column name and is null. Looks like postgres at least parses this correctly but I am not sure about other databases.

Would it be possible to add licence information for this project?