Fix build issue with KafkaDataset (#17418)
authorYong Tang <yong.tang.github@outlook.com>
Thu, 8 Mar 2018 07:22:47 +0000 (23:22 -0800)
committerGunhan Gulsoy <gunan@google.com>
Thu, 8 Mar 2018 07:22:47 +0000 (23:22 -0800)
commit37f6d224d69edd532197d615ace872933be5d74b
tree9176c61ab0124707cc8dd6c413b438afd261bb06
parentf9fb7e7736423f0bd416e1949e614d302c929709
Fix build issue with KafkaDataset (#17418)

* Fix build issue with KafkaDataset

This fix tries to address the issue raised in 17210 where
error of `NotFoundError: Op type not registered 'KafkaDataset' in binary.`
returned from kafka ops. The issue was that the inclusion of kafka ops
was removed due to the conflict merge from the other PR. This fix
fixes the issue.

This fix fixes 17210.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Change `import readers.Dataset` to `import dataset_ops.Dataset`,

due to the changes in some other places.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Fix library dependency issues in bazel

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Add dependency to bazel rules

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Add license to lib and pip package

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Remove unneeded changes in bazel

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Address review feedback

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Fix sanity check

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Add zlib dependency and include path

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Add copts in bazel to address the discrepancy in clang and gcc

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
tensorflow/contrib/BUILD
tensorflow/contrib/kafka/BUILD
tensorflow/contrib/kafka/kernels/kafka_dataset_ops.cc
tensorflow/contrib/kafka/ops/dataset_ops.cc [moved from tensorflow/contrib/kafka/ops/kafka_ops.cc with 100% similarity]
tensorflow/contrib/kafka/python/ops/kafka_dataset_ops.py
tensorflow/contrib/kafka/python/ops/kafka_op_loader.py [new file with mode: 0644]
tensorflow/tools/pip_package/BUILD
third_party/kafka/BUILD