[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-585427783 @mcvsubbu , I have addressed the review comments: Latest changes: - Move the realtime

[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-585427783 @mcvsubbu , I have addressed the review comments: Latest changes: - Move the realtime

[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-585427783 @mcvsubbu , I have addressed the review comments: Latest changes: - Move the realtime

[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-585427783 @mcvsubbu , I have addressed the review comments: Latest changes: - Move the realtime

[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576459692 > Related comment in PR 4954. I don't have an answer, but we do need to resolve it > >

[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576460048 > Also, is this supported in the SQL path? Yes it will be supported.

[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576464652 TODOs: (1) Add the new data type. (2) Extend Calcite SQL grammar and parser to support

[GitHub] [incubator-pinot] siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime)

siddharthteotia edited a comment on issue #4993: Support Text column type in Pinot (both offline and realtime) URL: https://github.com/apache/incubator-pinot/pull/4993#issuecomment-576460048 > Also, is this supported in the SQL path? Yes it will be supported. As noted in my comments