HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-531647803
Rebased with master as we merged #22282
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-529248383
Ping.
This is an
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-523211810
@jose-torres @gaborgsomogyi Could we take another round of review as I
addressed all
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-488107608
Ping again, as Spark+AI Summit 2019 in SF is end.
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-480483302
Ping.
This is an
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-477886815
I addressed all review comments so far: so please take another round of
review.
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-476865848
Updated. Could we have another round of review soon?
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-476828021
Requiring all partitions to be specified is not only having flexibility, but
also
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-476025207
Kindly ask to get another round of review.
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-473711699
retest this, please
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-473501308
Due to deal with function parameters with long param names, the indentation
may look
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-472842067
@tdas @zsxwing @jose-torres Could we consider start reviewing this one to
add this
HeartSaVioR commented on issue #23747: [SPARK-26848][SQL] Introduce new option
to Kafka source: offset by timestamp (starting/ending)
URL: https://github.com/apache/spark/pull/23747#issuecomment-470466337
IMHO [SPARK-23539](https://github.com/apache/spark/pull/22282) and this
13 matches
Mail list logo