Github user njayaram2 commented on the pull request:
https://github.com/apache/incubator-madlib/pull/44
@decibel Thank you for the comment. This is certainly useful, and I am
planning to implement this as part of sessionization-Phase 2 (I will put in the
link of the corresponding JIRA soon). The following are the changes I plan to
make based on your comment:
1) Have an optional "params" parameter which would include:
- **create_view** (T/F): default is **true**
- (TRUE) will create a view, instead of table.
- (FALSE) will create a table. The name for the view/table will be
specified in the output_table parameter.
- **output_all_columns** (T/F): default is **fault**.
- (FALSE) will output the partition columns, time_stamp column and a new
session ID column. The assumption is that the partition columns with the
time_stamp column will be sufficient to perform a join with the input table.
- (TRUE) will output all the columns in the table, along with the new
session_id column.
2) Other optional parameters such as **min_time** and **persist_null** can
also be specified in "params" (details of these will be updated in the JIRA).
Please let me know if you have any other comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
